Skip to content

Conversation

@Apisathan
Copy link

Made it so the player help can be translated, doesnt do the admin and other parts.

Set<String> permissions = plugin.getConfig().getConfigurationSection("command.player-help").getKeys(false);
for (String permission : permissions) {
if(!permission.contains("default") && !sender.hasPermission(subcommands.get(permission).getPermission())) continue;
for(String message : plugin.getConfig().getStringList("command.player-help."+permission)) {
Copy link
Owner

Choose a reason for hiding this comment

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

Please don't use a string list to define multiline messages, instead use on of the many different ways to define newlines in YAML

https://yaml-multiline.info/

# Trying to add new categories? This has been moved to categories.yml

#Default section doesn't require any permission
command:
Copy link
Owner

Choose a reason for hiding this comment

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

This should be with the other messages (though this system is going to redone at some point in the future anyway)

@LMBishop LMBishop linked an issue Mar 1, 2022 that may be closed by this pull request
3 tasks
@LMBishop LMBishop removed a link to an issue Mar 1, 2022
3 tasks
@LMBishop LMBishop linked an issue Mar 19, 2022 that may be closed by this pull request
2 tasks
@SrBedrock
Copy link

Any updates? 🤔

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.

Ability to configure /quests help messages

3 participants