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

feat: Client commands stubbing #1592

Merged
merged 2 commits into from
May 22, 2024

Conversation

Tiernan-Alderman
Copy link
Contributor

@Tiernan-Alderman Tiernan-Alderman commented May 22, 2024

FEAT: Implementing client commands

Description: Implemented a list of commands which are used by the client rather than the server. Essentially made an empty command for the server which parses the command through to the client, which then does its thing (this mostly includes things like emotes and a few config options)

Motivation and Context: Adding the ability to use these commands to continue fleshing out our current command palette and help command.

Type of Changes: Changes made to SlashCommandHandler.cpp to include additional command attributes for client commands, added a declaration in GMZeroCommands.h and definition in GMZeroCommands.cpp

How Has This Been Tested: Running commands in game, which triggers their respective emotes and or other functionality. Additionally, these client commands are listed in the /help menu for users to have access to them and get information as necessary.

Adding list of client commands provided to me by EmosewaMC
@EmosewaMC
Copy link
Collaborator

as a note for the testing, these commands have always worked and were never broken. This just prevents a "command not found" from being sent to the clients when one of these is used, and adds a help message for them.

@aronwk-aaron aronwk-aaron merged commit dea10c6 into DarkflameUniverse:main May 22, 2024
4 checks passed
@aronwk-aaron aronwk-aaron changed the title Client commands implementation feat: Client commands stubbing May 22, 2024
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.

3 participants