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

Send real_name in quote notification emails #7736

Merged
merged 1 commit into from
Jul 12, 2023
Merged

Conversation

BrickOzp
Copy link
Contributor

The username was used in the quote notification emails instead of real_name.

Fixes #7734

The username was used in the quote notification emails
instead of real_name.

Fixes #7734
@sbulen sbulen added this to the 2.1.5 milestone Apr 26, 2023
@live627
Copy link
Contributor

live627 commented Jun 14, 2023

looks like the relevant info is in MEMBERNAME and that we may need to fix the relevant language string instead.

@BrickOzp
Copy link
Contributor Author

No MEMBERNAME and QUOTENAME are different users
From the email template

Hello {MEMBERNAME},

You have been quoted in the post titled "{CONTENTSUBJECT}" by {QUOTENAME}, you can see the post here

@live627
Copy link
Contributor

live627 commented Jun 16, 2023 via email

@live627 live627 merged commit aea4824 into release-2.1 Jul 12, 2023
@jdarwood007 jdarwood007 deleted the quote_name branch July 17, 2023 01:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Notification emails should use display name in all circumstances
3 participants