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

Add parameter with_localizations to get_global_application_commands #2431

Closed
wants to merge 4 commits into from

Conversation

sandlotie
Copy link
Contributor

No description provided.

@github-actions github-actions bot added http Related to the `http` module. model Related to the `model` module. labels May 14, 2023
@kangalio
Copy link
Collaborator

This is a breaking change. Ideally the PR should be redesigned to not need a breaking change, but if impossible, it should be retargeted at the next branch

@kangalio
Copy link
Collaborator

Thank you!

@mkrasnitski
Copy link
Collaborator

I actually prefer the breaking changes - parameterizing the endpoint using two separate methods seems kinda weird. However, if that's what we're sticking with, you should add wrapper methods to Guild and PartialGuild that wrap the new method on GuildId.

@arqunis arqunis added the enhancement An improvement to Serenity. label May 18, 2023
@arqunis
Copy link
Member

arqunis commented May 18, 2023

@arqunis arqunis closed this May 18, 2023
@kangalio
Copy link
Collaborator

By the way @acdenisSK, you can also push commits onto PR branches. Would be a bit cleaner and possibly easier than manually merging

mkrasnitski added a commit to mkrasnitski/serenity that referenced this pull request May 21, 2023
mkrasnitski added a commit to mkrasnitski/serenity that referenced this pull request May 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An improvement to Serenity. http Related to the `http` module. model Related to the `model` module.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants