Skip to content

Conversation

@holzmaster
Copy link
Collaborator

No description provided.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 1, 2026

Hallo, hier kannst du jetzt deine tolle neue Funktion testen: https://discord.gg/pJjfF6n2ec

Copy link
Collaborator

@twobiers twobiers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Das Changeset als solches sieht ok für mich aus. Ich frag mich nur konzeptuell ob das überhaupt sinnvoll/notwendig ist zwischen modcommands und normalen commands zu unterscheiden.

Ich finde commands sollten permissions / role restrictions haben. Die hilfe könnte dann entsprechend auch mit den permissions filtern.

Irgendwie macht das hier ziemlich viel komplexität auf und lädt bugs ein.

@holzmaster
Copy link
Collaborator Author

Hab den constraint aufgelöst, dass mod-commands und normale commands dieselben namen haben können.

Brauchen wir da überhaupt noch verschiedene präfixe?

@twobiers
Copy link
Collaborator

twobiers commented Feb 1, 2026

Brauchen wir da überhaupt noch verschiedene präfixe?

Für die Gewohnheit, ja. Aber technisch ist das nicht mehr notwendig denke ich.

@holzmaster
Copy link
Collaborator Author

Schau jetzt nochmal drüber

Copy link
Collaborator

@twobiers twobiers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Das gefällt mir signifikant besser, nice

@holzmaster holzmaster merged commit a04185d into master Feb 1, 2026
6 checks passed
@holzmaster
Copy link
Collaborator Author

Gucken wir mal, obs funktioniert

@holzmaster holzmaster deleted the commands-fix branch February 1, 2026 21:59
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.

3 participants