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.

Sign up for GitHub

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

Jump to bottom

ENH: Token replacement for email template #1135

Merged
merged 2 commits into from
Oct 29, 2024
Merged

ENH: Token replacement for email template #1135

merged 2 commits into from
Oct 29, 2024

Conversation

johnhenley
Copy link
Collaborator

Description of PR...

Modernizes token replacement for email notifications to use DNN API and entity-based IPropertyAccess

Changes made

  • Add some new properties to be (edit user link, private message, etc.)
  • Add new legacy mappings
  • Add new [FORUMAUTHOR:*] token support to mirror [FORUMUSER:*] but in context of an author
  • Add new [FORUMAUTHORUSER:*] token support to mirror [USER:*] but in context of an author
  • Add new [FORUMAUTHORMEMBERSHIP:*] token support to mirror [MEMBERSHIP:*] but in context of an author
  • Add new [FORUMAUTHORPROFILE:*] token support to mirror [PROFILE:*] but in context of an author
  • Prevent HTML tags in email subject (BUG: email notification subject includes html e.g., span tag #1127)

How did you test these updates?

Local install

PR Template Checklist

  • Fixes Bug
  • Feature solution
  • Other
  • Requires documentation updates
  • I've updated the documentation already

Please mark which issue is solved

Close #1127
Close #1128

@johnhenley johnhenley added enhancement New feature or request technical debt Issue that doesn't directly affect usability but will improve technical debt posture themes Dnn Community Forums Themes labels Oct 29, 2024
@johnhenley johnhenley added this to the 08.02.00 milestone Oct 29, 2024
@johnhenley johnhenley self-assigned this Oct 29, 2024
@johnhenley johnhenley merged commit 946f0da into DNNCommunity:dev Oct 29, 2024
1 check passed
@johnhenley johnhenley deleted the dev82/email-token-replacement-remove-html-subject-1127-1128 branch October 29, 2024 21:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@johnhenley johnhenley

Labels
enhancement New feature or request technical debt Issue that doesn't directly affect usability but will improve technical debt posture themes Dnn Community Forums Themes
Projects
None yet
Milestone
08.02.00
Development

Successfully merging this pull request may close these issues.

ENH: modernize email token replacement BUG: email notification subject includes html e.g., span tag
1 participant
@johnhenley