Feature Request/Idea: Enable html for all email notifications
Nobody has claimed this yet.
- Dominant language
- Java
- Stars
- 1.1k
- Forks
- 564
- Avg merge
- 2d 2h
- Merged PRs (30d)
- 29
Description
Overview of the Feature Request
Make layout of email notifications more consistent (and avoid unnecessarily long links) by enabling html for all notifications.
What kind of user is the feature intended for?
(Example users roles: API User, Curator, Depositor, Guest, Superuser, Sysadmin)
All
What inspired the request?
User feedback.
What existing behavior do you want changed?
This refers in particular to the following notifications:
notification.email.grantFileAccess=Access granted for files in dataset: {0} (view at {1} ).
notification.email.rejectFileAccess=Your request for access was rejected for the requested files in the dataset: {0} (view at {1} ).
notification.email.createDataverse=Your new dataverse named {0} (view at {1} ) was created in {2} (view at {3} ).
notification.email.createDataset=Your new dataset named {0} (view at {1} ) was created in {2} (view at {3} ).
notification.email.wasSubmittedForReview={0} (view at {1} ) was submitted for review to be published in {2} (view at {3} ).
notification.email.wasReturnedByReviewer={0} (view at {1} ) was returned by the curator of {2} (view at {3} ).
notification.email.wasPublished={0} (view at {1} ) was published in {2} (view at {3} ).
notification.email.publishFailedPidReg={0} (view at {1} ) in {2} (view at {3} ) could not be published
notification.email.assignRole=You are now {0} for the {1} "{2}" (view at {3} ).
notification.email.revokeRole=One of your roles for the {0} "{1}" has been revoked (view at {2} ).
On the other hand, this has already been introduced for the following notifications:
notification.email.datasetWasCreated=Dataset "<a href = "{0}">{1}</a>" was just created by {2} in the {3} collection.
notification.email.requestedFileAccess=You have requested access to a file(s) in dataset "<a href = "{0}">{1}</a>".
notification.email.datasetWasMentioned=Hello {0},<br><br> The {1} has just been notified that the {2}, <a href=''{3}''>{4}</a>, {5} "<a href=''{6}/dataset.xhtml?persistentId={7}''>{8}</a>" in this repository.
Any brand new behavior do you want to add to Dataverse?
Maybe a general setting to switch html on/off for all email notifications?
Any open or closed issues related to this feature request?
Disable Email Notification Options #9978
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating the notification.email keys listed in the issue and compare them with the existing HTML-enabled examples such as datasetWasCreated and requestedFileAccess. Review related issue #9978 for context, then verify that the affected notifications have consistent HTML link formatting and that the intended all-notification behavior is covered.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- html, java
- Domain
- backend
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100