Conversation
The first use case is to avoid returning a JSF/xhtml URL to the SPA.
|
📦 Pushed preview images as 🚢 See on GHCR. Use by referencing with full name as printed above, mind the registry name. |
|
Another thing to consider for this PR: the notification text that is currently being returned is for emails, not for the Notifications tab in the Account Page. For example, for a new role, the API returns But for the Notifications tab, we see a different message:
So to support this, as Jim suggested, the dynamic values to insert into the translation text can be passed as additional parameters. |
|
FWIW: To handle mail, I think we'd need a global (truly global or per-user) setting for preferred UI rather than a param on specific api calls - otherwise taking the same action from different UIs will result in different messages/one user might get emails pointing to different UIs if their colleagues make changes from different UIs. It would also be nice to get the URL generation out of the properties files, i.e. by keeping the templates for dataverse/dataset/datafile page generation for different UIs in the db (which might mean new UIs could register their page links some day). |
|
Closing. This was just an experiment. I got farther in another experiment at https://github.com/IQSS/dataverse/tree/11648-notification-patterns As mentioned at #11673 (comment) @GPortas is taking a swing at this. We'll compare notes afterward. |

What this PR does / why we need it:
This is just a draft PR to help us start thinking about how to approach this issue:
The first use case is to avoid returning a JSF/xhtml URL to the SPA.
Which issue(s) this PR closes:
Special notes for your reviewer:
Suggestions on how to test this:
Does this PR introduce a user interface change? If mockups are available, please link/include them here:
Is there a release notes update needed for this change?:
Additional documentation: