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

Add config file for better command aliases #9971

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Machine-Maker
Copy link
Member

@Machine-Maker Machine-Maker commented Nov 25, 2023

Adds a new file, command-aliases.yml into our config folder for defining simple aliases for commands that work with tab completion and permissions. Schema is as follows

alias1: some-command

alias2:
  - some
  - command

alias3:
  target: some-command
  permission: some.permission

alias4:
  target:
    - some
    - command
  permission: some.permission

The default permission behavior is to combine all of the permission requirements at all of the nodes along the target path and assign that to the alias node. But that can be changed to a custom permission in the config.


Closes #9214
Related #9458


Will be merged into the paper config patch just before merging.

@Machine-Maker Machine-Maker requested a review from a team as a code owner November 25, 2023 20:33
@YanisBft
Copy link

YanisBft commented Dec 4, 2023

Great work, will be very helpful! Do you plan on deprecating Bukkit command aliases? Or still support it officially

456dev added a commit to 456dev/paper-docs that referenced this pull request Dec 12, 2023
@Machine-Maker
Copy link
Member Author

Do you plan on deprecating Bukkit command aliases?

I don't think we have to, its pretty barebones as-is, and just won't get updates (like it hasn't gotten updates in years anyways).

@Huge-mistake
Copy link

Bump, is there still a chance to merge this PR? (╯°□°)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Awaiting review
Development

Successfully merging this pull request may close these issues.

Aliases for commands defined in commands.yml don't properly TAB complete
3 participants