-
Notifications
You must be signed in to change notification settings - Fork 24
Modcommand fix #574
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
Modcommand fix #574
Conversation
|
Hallo, hier kannst du jetzt deine tolle neue Funktion testen: https://discord.gg/pJjfF6n2ec |
twobiers
left a comment
There was a problem hiding this 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.
|
Hab den constraint aufgelöst, dass mod-commands und normale commands dieselben namen haben können. Brauchen wir da überhaupt noch verschiedene präfixe? |
Für die Gewohnheit, ja. Aber technisch ist das nicht mehr notwendig denke ich. |
|
Schau jetzt nochmal drüber |
twobiers
left a comment
There was a problem hiding this 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
|
Gucken wir mal, obs funktioniert |
No description provided.