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(ApplicationCommandRegistries): add the right ids in the right place #716

Merged
merged 6 commits into from
Jan 13, 2024

Conversation

vladfrangu
Copy link
Member

Doesn't break anything funnily enough but its technically wrong

Doesn't break anything funnily enough but its technically wrong
kyranet
kyranet previously approved these changes Jan 12, 2024
@vladfrangu vladfrangu marked this pull request as draft January 12, 2024 22:53
@vladfrangu
Copy link
Member Author

@sapphiredev pack this pls

@sapphiredev
Copy link

sapphiredev bot commented Jan 12, 2024

Heya @vladfrangu, I've started to run the deployment workflow on this PR at 2eb6ec8. You can monitor the build here!

@vladfrangu vladfrangu marked this pull request as ready for review January 12, 2024 23:21
@sapphiredev
Copy link

sapphiredev bot commented Jan 12, 2024

Hey @vladfrangu, I've released this to NPM. You can install it for testing like so:

npm install @sapphire/framework@pr-716

@vladfrangu
Copy link
Member Author

@sapphiredev pack this pls

@sapphiredev
Copy link

sapphiredev bot commented Jan 12, 2024

Heya @vladfrangu, I've started to run the deployment workflow on this PR at ddccc23. You can monitor the build here!

@favna favna merged commit 34193ef into main Jan 13, 2024
8 checks passed
@favna favna deleted the fix/correctly-set-ids-to-right-places-for-bulk-overwrite branch January 13, 2024 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants