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

Allow Disabling /cosmetics Command #164

Draft
wants to merge 2 commits into
base: remapped
Choose a base branch
from

Conversation

DebitCardz
Copy link
Contributor

Select the option(s) that best describes this PR:

  • Feature implementation

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

Allows easy overrides when implementing custom state.

Check that:

  • Any new classes, public methods and/or properties are properly documented with JavaDocs
  • Syntax and style are consistent with existing code
  • Any replaced method isn't deleted, but rather labeled as deprecated

Note In the case where the new method has the exact same signature as the method it's replacing, mention above that the old method was deleted.

@DebitCardz DebitCardz marked this pull request as draft January 25, 2025 21:01
@DebitCardz
Copy link
Contributor Author

Weird, un-registering from the command map doesn't seem to take it out and let me override.
Will need to look more into later, overriding the aliases works though.

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

Successfully merging this pull request may close these issues.

1 participant