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

Review list of commands to display in the Thread's composer #4474

Closed
2 tasks
daniellekirkwood opened this issue Nov 16, 2021 · 2 comments
Closed
2 tasks

Review list of commands to display in the Thread's composer #4474

daniellekirkwood opened this issue Nov 16, 2021 · 2 comments
Assignees
Labels
A-Threads An issue for the Threads project X-Needs-Design May require input from the design team Z-ThreadsP0 Issue to be released with the Threads MVP

Comments

@daniellekirkwood
Copy link
Contributor

daniellekirkwood commented Nov 16, 2021

There are items in the /command list that are only applicable to the room and so it does not make sense to have them appear in the Thread.

Here is a list of commands and the proposal for which to remove from the Threads composer

  • Remove the commands from the list (When a user types "/" a list appears, this should only contain the actions that the user can take)
  • When a user tries to use a command that is not available in the Thread, show an error message that says:

Command error
This command is not permitted in a thread
OK

@daniellekirkwood daniellekirkwood added the T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements label Nov 16, 2021
@daniellekirkwood daniellekirkwood added A-Threads An issue for the Threads project Z-ThreadsP0 Issue to be released with the Threads MVP and removed T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements labels Nov 16, 2021
@daniellekirkwood
Copy link
Contributor Author

@janogarcia I've added a solution into the issue info here, please let us know if there's anything that should change

@janogarcia
Copy link

janogarcia commented Nov 29, 2021

I'd suggest tweaking the copy to make it a bit more helpful, by telling the users how to move forward.

Unavailable command
This command is not available in threads. Try running it in the room instead.
OK

...or something along the lines. What do you think, @daniellekirkwood?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Threads An issue for the Threads project X-Needs-Design May require input from the design team Z-ThreadsP0 Issue to be released with the Threads MVP
Projects
None yet
Development

No branches or pull requests

3 participants