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

request: Add ApplicationCommandRegistries BulkOverwrite Event Listener #821

Open
1 task done
ryouzue opened this issue Feb 22, 2025 · 2 comments
Open
1 task done

Comments

@ryouzue
Copy link

ryouzue commented Feb 22, 2025

Is there an existing issue or pull request for this?

  • I have searched the existing issues and pull requests

Feature description

May serve for other use cases but I want to use it to be able to replace the default [INFO] ApplicationCommandRegistries(BulkOverwrite) + ApplicationCommandRegistries[command.name] that I get when I boot up my bot

NVIDIA_Overlay_PuAllcywsm.mp4

Desired solution

Check out thread (ID: 1342779852142546974) on Discord for more information

https://canary.discord.com/channels/737141877803057244/1342779852142546974

Alternatives considered

Check out thread (ID: 1342779852142546974) on Discord for more information

Additional context

No response

@vladfrangu vladfrangu changed the title request: Add ApplicationCommandRegistries Event Listener request: Add ApplicationCommandRegistries BulkOverwrite Event Listener Feb 22, 2025
@ryouzue
Copy link
Author

ryouzue commented Mar 1, 2025

any information as for when this is going to be implemented?

@favna
Copy link
Member

favna commented Mar 1, 2025

Fraid to say that Vladdy, Kyra and I are all heavily starved for time as we all have full time jobs.

That said if you make a pull request we'll be more than glad to review, merge, and release it. The general gist I think is moving the log to an optional listener (https://github.com/sapphiredev/framework/tree/main/src/optional-listeners/application-command-registries-listeners) that is loaded based on a client option (default to true if bulkoverwrite is chosen, otherwise default false). Then where the code for the log is now, you instead emit to the listener.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants