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
Conversation
@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 |
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 😢. |
Co-authored-by: Noel <buechler.noel@outlook.com>
Co-Authored-By: Suneet Tipirneni <77477100+suneettipirneni@users.noreply.github.com>
e6aa42b
to
b186cc6
Compare
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. |
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: