-
Notifications
You must be signed in to change notification settings - Fork 11
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: Add /help command #2984
feat: Add /help command #2984
Conversation
Windows and Mac build successfull in Unity Cloud! You can find a link to the downloadable artifact below.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Congrats on your first PR! 👏 👏 👏
Explorer/Assets/Scripts/Global/Dynamic/ChatCommands/GoToChatCommand.cs
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🟢 Feature verified on Windows and Mac. The /help
command is working as expected displaying accurate data.
Smoke test was performed covering:
- Existing command functionality
- Teleport using chat and maps
- Changing wereables
- Social interactions --> Emote sync, wereables sync, chats, passport
- Side Bar functionality
- Profile info edit
- Scenes/world visited: Doll House, Casa Roustan, world Pravus, World Olavra, Metadynelabs
f0f9c2a
to
03fc8fa
Compare
What does this PR change?
Adds a /help command with a small refactor to remove deferred command creation and simplify the code.
fix:#2567
How to test the changes?
Our Code Review Standards
https://github.com/decentraland/unity-renderer/blob/master/docs/code-review-standards.md