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 GET prefix for ApplicationCommands bucketID #1193

Merged
merged 5 commits into from Jun 19, 2022

Conversation

plally
Copy link
Contributor

@plally plally commented Jun 1, 2022

This addresses #1173

@FedorLap2006 FedorLap2006 linked an issue Jun 1, 2022 that may be closed by this pull request
@FedorLap2006 FedorLap2006 self-requested a review June 1, 2022 21:39
@FedorLap2006 FedorLap2006 added the fix Pull requests and issues related to bug fixes and structural inconsistencies label Jun 1, 2022
restapi.go Outdated Show resolved Hide resolved
@FedorLap2006
Copy link
Collaborator

Sorry made you wait.

plally and others added 2 commits June 14, 2022 19:15
Co-authored-by: Fedor Lapshin <fe.lap.prog@gmail.com>
@plally plally requested a review from FedorLap2006 June 14, 2022 23:22
@FedorLap2006
Copy link
Collaborator

LGTM, however there were some changes affecting this particular line, so please resolve the conflicts.

@FedorLap2006 FedorLap2006 merged commit a7f0378 into bwmarrin:master Jun 19, 2022
@FedorLap2006
Copy link
Collaborator

Thanks for your contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix Pull requests and issues related to bug fixes and structural inconsistencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Wrong rate limit bucket for commands endpoint
2 participants