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

fix: follow discord-api-types package's type #7267

Closed
wants to merge 1 commit into from
Closed

fix: follow discord-api-types package's type #7267

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Jan 14, 2022

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

After this, CommandInteractionOptionResolver#getX only accepts discord-api-types's type. It throws an error because old thing is ALL UPPERCASE, while discord-api-types using PascalCase.

Status and versioning classification:

  • I know how to update typings and have done so, or typings don't need updating

@ghost ghost changed the title fix: follow type of discord-api-types package fix: follow discord-api-types package's type Jan 14, 2022
@ImRodry
Copy link
Contributor

ImRodry commented Jan 14, 2022

This change was already proposed on another PR and rejected by @iCrawl so this will probably have the same faith

@iCrawl iCrawl closed this Jan 14, 2022
@iCrawl iCrawl added this to the discord.js v14 milestone Jan 14, 2022
@ghost
Copy link
Author

ghost commented Jan 14, 2022

This change was already proposed on another PR and rejected by @iCrawl so this will probably have the same faith

But it's fact that those methods are broken. how to fix the problem without following dapi-types package's way?

@iCrawl
Copy link
Member

iCrawl commented Jan 14, 2022

Not use -dev is a way.

If you need interaction locales theres also 13.6.0

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.

3 participants