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 playername tabcompleting to the root money command. #41

Merged
merged 7 commits into from
Jul 25, 2023

Conversation

ewof
Copy link

@ewof ewof commented Jul 25, 2023

adds player names to /money tab complete
fixes null controller exception when doing /money set <player> <amount> from console
changes ? to use <> for required and [] for optional as is standard and shows player as an optional parameter for /money

ewof added 3 commits July 24, 2023 19:49
update placeholders to use <> as required and [] as optional
make ? command show player as optional argument for /money
update readme to new ? command
ewof and others added 2 commits July 25, 2023 00:19
src/main/java/com/iConomy/entity/Players.java Outdated Show resolved Hide resolved
src/main/java/com/iConomy/entity/Players.java Outdated Show resolved Hide resolved
ewof and others added 2 commits July 25, 2023 00:29
Co-authored-by: LlmDl <LlmDlio@gmail.com>
Co-authored-by: LlmDl <LlmDlio@gmail.com>
@ewof ewof requested a review from LlmDl July 25, 2023 14:46
@LlmDl LlmDl changed the title Players tab complete Add playername tabcompleting to the root money command. Jul 25, 2023
@LlmDl LlmDl merged commit bfc6868 into iconomy5legacy:master Jul 25, 2023
LlmDl added a commit that referenced this pull request Jul 26, 2023
  - Add playername tabcompleting to the root money command, courtesy of
ewof with PR #41.
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.

2 participants