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

email Dataverse Contact / authenticated users #3545

Closed
pameyer opened this issue Dec 21, 2016 · 7 comments
Closed

email Dataverse Contact / authenticated users #3545

pameyer opened this issue Dec 21, 2016 · 7 comments

Comments

@pameyer
Copy link
Contributor

pameyer commented Dec 21, 2016

For authenticated users, it would be helpful if the username was included in the email sent to the dataverse contact.
Same behavior observed for "support" menu item.

@pdurbin
Copy link
Member

pdurbin commented Dec 22, 2016

@pameyer are you talking about the "Welcome to Dataverse!" email that is sent to the person who just created an account? Here is the text: https://github.com/IQSS/dataverse/blob/v4.6/src/main/java/Bundle.properties#L440

I'm not sure what you mean by "dataverse contact". A user does not necessarily become a contact for a dataverse. The user could just have a ready only role such as "Member", for example.

@pameyer
Copy link
Contributor Author

pameyer commented Dec 22, 2016

@pdurbin - there are buttons in the GUI for "contact support" / ~"message to dataverse contact". These both send emails; but neither of those emails includes the source username

@pdurbin
Copy link
Member

pdurbin commented Dec 22, 2016

Ah you're talking about:

    if (isLoggedIn() && userMessage!=null) {
        mailService.sendMail(loggedInUserEmail(), email, getMessageSubject(), userMessage);

... over at https://github.com/IQSS/dataverse/blob/v4.6/src/main/java/edu/harvard/iq/dataverse/SendFeedbackDialog.java#L212

@mheppler is actually working on code related to this over in d800944 for #3036. Maybe there's a chance to coordinate in that branch.

@pdurbin
Copy link
Member

pdurbin commented Jan 6, 2017

As @mheppler points out, this is related to the "who wrote" aspect of #1916.

@pdurbin
Copy link
Member

pdurbin commented Jan 6, 2017

#2882 is related.

@mheppler
Copy link
Contributor

mheppler commented Jan 6, 2017

@pameyer this issue you opened looks to be similar to Email Dataset/Dataverse Contact: No context info given in the email correspondance received #1916.

Please clarify any differences between these issues (e.g. "username") in a new comment #1916, and then close issue as a duplicate. Thank you.

@pameyer
Copy link
Contributor Author

pameyer commented Jan 6, 2017

closing in favor of #1916; will re-open if necessary

@pameyer pameyer closed this as completed Jan 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants