kord-extensions icon indicating copy to clipboard operation
kord-extensions copied to clipboard

defaultingBoolean looking weird on the help command

Open Unicorn7141 opened this issue 3 years ago • 3 comments

Description

I was trying to make a shop command which allows people to add an item and decide whether it's limited to quantity X or not, using a defaultingBoolean with a default false. When I checked the command using help it showed the following: [limited?: yes/no=false] which could be quite confusing.

Versions

KordEx version: 1.5.0-SNAPSHOT

Reproduction

Code Examples

class Test : Arguments() {
    val bool by defaultingBoolean("bool", "The boolean to print", defaultValue = false)
}

Suggestions

Change the help command to show "yes" or "no" according to true or false

Media

Click to expans

image

Unicorn7141 avatar Sep 20 '21 06:09 Unicorn7141

Hello, and thanks for opening an issue! As this is the first time you've created an issue on this repository, we'd just like to offer you a warm welcome to the project, and the following pointers:

  • Most importantly, all issues must adhere to our Code of Conduct. Please give it a quick read if you haven't already.

  • While our team is passionate about the projects we've created here, we're all volunteers. Please don't be offended if it takes time for us to get to your issue - we'll be here as soonas we can be!

  • Please provide as much information as possible when asking a question, reporting a problem, or submitting a feature request. This will help us to address your issue quickly and efficiently. If you forgot to add some information, no worries - feel free to edit the issue and add anything you missed!

    Thanks for contacting us! If you have any further questions, please feel free to join us on Discord in the #dev-kotdis channel (or #kordex-discussion for Kord Extensions projects), or to contact a staff member directly.

boring-cyborg[bot] avatar Sep 20 '21 06:09 boring-cyborg[bot]

Having looked into this, defaulting converters will show the precise value you've provided as the default value there. I'm not sure special-casing Boolean converters for this is the way to go, maybe some kind of translation method? I'm really not sure - I'll have to think about it.

gdude2002 avatar Sep 22 '21 09:09 gdude2002

So far, I haven't been able to come up with a smart solution for this.

gdude2002 avatar Oct 12 '21 10:10 gdude2002