Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(ApplicationCommand): add support for min and max values #6855

Merged
merged 6 commits into from Nov 11, 2021

Conversation

ckohen
Copy link
Member

@ckohen ckohen commented Oct 18, 2021

Please describe the changes this PR makes and why it should be merged:

Adds support for number and int types max and min values

Typings are not yet updated due to a potential issue with choices and min / max that I will be noting upstream

Status and versioning classification:

  • Code changes have been tested against the Discord API, or there are no code changes
  • This PR changes the library's interface (methods or parameters added)

src/structures/ApplicationCommand.js Show resolved Hide resolved
@samarmeena
Copy link
Contributor

@ckohen a question.

About a week has passed since upstream PR were merged, can you tell me why typings are not yet available?

note: IK rebase isn't required but can you please rebase your fork. Thanks

@ckohen
Copy link
Member Author

ckohen commented Nov 8, 2021

Unfortunately the question in the upstream PR was not answered, so I think I'm just going to have to make a decision and leave the typings less strict 😢.

typings/index.d.ts Outdated Show resolved Hide resolved
typings/index.d.ts Outdated Show resolved Hide resolved
@samarmeena
Copy link
Contributor

Assuming this PR is completed and it's been over a week since upstream PR merged. May I ask what's stopping this PR to merge into main?

Curious to try this update.

@iCrawl iCrawl merged commit c30a818 into discordjs:main Nov 11, 2021
@ckohen ckohen deleted the options-max-min branch November 11, 2021 18:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

9 participants