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] Feature/livechat hide email #8149

Merged
merged 13 commits into from
Dec 8, 2017

Conversation

icosamuel
Copy link
Contributor

@icosamuel icosamuel commented Sep 14, 2017

@RocketChat/core
@RocketChat/livechat
Closes #6612

New Livechat_show_agent_email config in the livechat tab of the web administration page as well as the Livechat management page, translated in en and fr.

The emails are not loaded from the database when we query for agent data, so they are never sent to the client.

@CLAassistant
Copy link

CLAassistant commented Sep 14, 2017

CLA assistant check
All committers have signed the CLA.

@RocketChat RocketChat deleted a comment Sep 14, 2017
@RocketChat RocketChat deleted a comment Sep 14, 2017
@engelgabriel engelgabriel added this to the 0.60.0 milestone Sep 18, 2017
@RichGriese
Copy link

RichGriese commented Sep 23, 2017

I just set up a Beta Rocket & am playing with LiveChat. In testing the operator email is showing. Is it too new to see this fix/mod in the beta, or is there some other issue? I see that the issues in still open. But it looks like the chatter indicates it's been done. I guess this is my intro in follow an issue through the process. Cheers!

@icosamuel
Copy link
Contributor Author

Hey @RichGriese! glad to see youre interested in this feature as well!

Unfortunately, I don't think we will see this in the next few releases, since it's going into the 0.60 feature bag, and the rocketchat pipeline is still making release candidates for 0.59

What I personally do to cherry pick the features that are not out yet, is to keep my own version of the project. If you have the resources, you can build the project after integrating this branch, or just build the project on my develop branch ;)

Hope this helps, I cant wait to get this feature out in the open!

@icosamuel
Copy link
Contributor Author

@sampaiodiego I realize that this is a minor feature, and that you might have a lot of different priorities.
With that said, may I request a review? I consider my work done and would like to close this PR.
Thanks for your time and help :)

@sampaiodiego
Copy link
Member

hi @icosamuel .. thank you very much for another contribution.

indeed it was a minor feature I was able to test between other tasks. =)

so far it looks good, but I noticed it is still sending agent's email over the wire (but correctly not showing it depending on the setting). what you think on preventing the email field to be sent to the client instead of just not showing it? TIA

@alexweissman
Copy link

alexweissman commented Oct 4, 2017

It would be ideal avoid sending the agent's email over the wire, but simply hiding it for now would be acceptable for the sake of getting this patched into the next release. It's a fairly significant privacy issue, especially for those of us with agents who didn't know that their email address would become public information in the context of livechat.

I'd say, merge this as-is for the 0.59 release, then update to avoid sending the email to the client at all in 0.60.

@icosamuel
Copy link
Contributor Author

sure, now that I'm more comfortable with the project, I can more easily change it's behavior to be on the server side.

It'll take some time though, as the requirements are kind of met on our side, so my time mostly goes on other tasks :/

@alexweissman
Copy link

@icosamuel the great tragedy of open source 😛

@icosamuel icosamuel changed the title [NEW] Feature/livechat hide email WIP: [NEW] Feature/livechat hide email Oct 25, 2017
@icosamuel icosamuel changed the title WIP: [NEW] Feature/livechat hide email [NEW] Feature/livechat hide email Oct 25, 2017
@icosamuel
Copy link
Contributor Author

Finally, I've found some time to sort this issue out! As of now, the emails do not even get out of the database if the setting is active.

This approach leaves us with much simpler changes and an elegant solution which does not even involve the Livechat client!

@rodrigok rodrigok merged commit e72e08e into RocketChat:develop Dec 8, 2017
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.

8 participants