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

[NEW] Adds preference to one-click-to-direct-message and basic functionality #7564

Merged
merged 3 commits into from
Jul 25, 2017

Conversation

gdelavald
Copy link
Contributor

@RocketChat/core

Closes #7436

This feature adds a preference to users who would prefer to skip opening the flex tab with personal information when clicking on a username/card and goes straight into direct message mode.

@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-7564 July 24, 2017 21:34 Inactive
@@ -330,6 +330,8 @@
"Clear_all_unreads_question": "Clear all unreads?",
"Click_here": "Click here",
"Click_here_for_more_info": "Click here for more info",
"Click_Direct_Message": "Click to Create Direct Message",
"Click_Direct_Message_Description": "Skip opening profile tab and goes straight to conversation",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Either Skips opening ... and goes ... or Skip opening ... and go ... -- I prefer the former.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"Skip opening profile tab, instead go straight to conversation" :D

Copy link
Member

@sampaiodiego sampaiodiego left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Clicking on myself gives me an "invalid user" error

@gdelavald
Copy link
Contributor Author

@sampaiodiego what do you think should be the behavior when clicking on yourself or for users that don't have direct channel permission?

@sampaiodiego
Copy link
Member

@gdelavald when clicking on yourself or for users that don't have direct channel permission I think we should show the profile panel, like the old/default behavior.

@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-7564 July 25, 2017 13:50 Inactive
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-7564 July 25, 2017 15:47 Inactive
@rodrigok
Copy link
Member

IMHO this should be an admin option instead of per user. The use case right now is to change the chat behavior to all users. We can add a sub setting in the future to change the default behavior per user.

@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-7564 July 25, 2017 18:36 Inactive
@rodrigok rodrigok dismissed stale reviews from marceloschmidt and sampaiodiego July 25, 2017 21:50

Fixed

@rodrigok rodrigok added this to the 0.58.0 milestone Jul 25, 2017
@rodrigok rodrigok merged commit c7604e2 into develop Jul 25, 2017
@rodrigok rodrigok deleted the one-click-to-direct-message branch July 25, 2017 22:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

Create an option to go straight to direct message when clicking on a user's name
6 participants