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

Fix: e2e password visible on always-on alert message. #12139

Merged
merged 4 commits into from
Sep 26, 2018

Conversation

Hudell
Copy link
Contributor

@Hudell Hudell commented Sep 24, 2018

This PR removes the password from the always-on alert message, so that the user will only see it when clicking on the alert.

@Hudell Hudell requested a review from rodrigok September 24, 2018 16:51
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-12139 September 24, 2018 16:51 Inactive
@Hudell Hudell added this to the 0.70.0 milestone Sep 24, 2018
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-12139 September 24, 2018 17:43 Inactive
@@ -547,6 +547,7 @@
"clear_history": "Clear History",
"Click_here": "Click here",
"Click_here_for_more_info": "Click here for more info",
"Click here to view and copy your password.": "Click here to view and copy your password.",
Copy link
Contributor

Choose a reason for hiding this comment

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

Shouldn't this be at least _ separated for consistency?

@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-12139 September 25, 2018 14:59 Inactive
@rodrigok rodrigok changed the title [FIX] e2e password visible on always-on alert message. Fix: e2e password visible on always-on alert message. Sep 26, 2018
@rodrigok rodrigok merged commit 9c0e687 into develop Sep 26, 2018
@rodrigok rodrigok deleted the remove-e2e-password-from-alert branch September 26, 2018 16:53
This was referenced Sep 28, 2018
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.

5 participants