Skip to content

Conversation

@ghost
Copy link

@ghost ghost commented Jul 17, 2015

To provide the recipient with the sender's name, add a required name field. If the user is already logged in, the user's name is inputted automatically, depending on the display name setting in admin area (username or real name).

To provide the recipient with the sender's name, add a required name field. If the user is already logged in, the user's name is inputted automatically, depending on the display name setting in admin area (username or real name).
@ghost ghost mentioned this pull request Jul 17, 2015
Added authentication check in line 76
@ecki
Copy link
Contributor

ecki commented Jul 24, 2015

I would recommend to always add the users's profile URL to the mail when the user is logged in. So you can make the name field editable, but you can still see what account was used (same is true for sender email modifications).

@ghost
Copy link
Author

ghost commented Jul 24, 2015

I totally agree to this proposal. Also, I would change the mail layout to the following:

++++++++++++++++++++++++++++++++++++++++++++++++++++++
New email via [site_name] - [site_url]

From: [Sender's name]
Email: [Sender's email]
Profile URL: [Sender's profile url]
Date: [Exact transmission date]

Subject: [Subject of email]

Message:
[Text of email]
++++++++++++++++++++++++++++++++++++++++++++++++++++++

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.

3 participants