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 a few command issues #151

Merged
merged 1 commit into from
Dec 10, 2024
Merged

Fix a few command issues #151

merged 1 commit into from
Dec 10, 2024

Conversation

Lyfts
Copy link
Member

@Lyfts Lyfts commented Dec 10, 2024

Blood magic overrides the lang key for the default /help commands usage with its own https://github.com/GTNewHorizons/BloodMagic/blob/2e95450cf0601c30e82126614d86b59ad02f09f0/src/main/resources/assets/alchemicalwizardry/lang/en_US.lang#L349. I can't imagine that it's actually intended but to fix it I've just hardcoded the help subcommand to be /insert_parent_command help which will also prevent any future issues.

Closes GTNewHorizons/GT-New-Horizons-Modpack#18323

The ConfigGroup class that the rank config ui uses only support periods and underscores as special characters in the name which caused some botania commands (/botania-open & /botania-share) to be saved with incorrect permission nodes. This is fixed by simply replacing any special character that is not a period or underscore in the command permissions with an underscore.

Closes GTNewHorizons/GT-New-Horizons-Modpack#18322

Signed-off-by: Lyfts <127234178+Lyfts@users.noreply.github.com>
@Lyfts Lyfts requested a review from a team December 10, 2024 16:26
@Dream-Master Dream-Master merged commit 3780a4a into master Dec 10, 2024
1 check passed
@Dream-Master Dream-Master deleted the fix-command-problems branch December 10, 2024 16:33
Dream-Master pushed a commit that referenced this pull request Dec 10, 2024
Signed-off-by: Lyfts <127234178+Lyfts@users.noreply.github.com>
(cherry picked from commit 3780a4a)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants