search

Rothamsted-Ecoinformatics / farm_rothamsted

Custom farmOS features for Rothamsted Research.
GNU General Public License v2.0
6 stars 1 forks source link

E-mail alerts: Extend notification dashboard #643

Closed aislinnpearson closed 3 months ago

aislinnpearson commented 6 months ago

An initial version of the notifications dashboard was created when we pushed the initial version of the e-mail alerts (issue #597), but it currently has limited options (see screen shot below). We discussed extending this so there was a general tab, one for experiments and one for proposals. The idea was essentially that you can choose which experiments or proposals you follow, which in future could also be extended to other assets or even taxonomy terms.

I've attached a PDF here with a draft following our initial conversations, but this will probably have to be discussed again once we implement #605 in milestone 2.20.0

E-mail Alerts - Example Notifications Dashboard.pdf

image

paul121 commented 4 months ago

Okay - I've simplified some of this notification dashboard to only include the changes for the "General" tab. Specifically adding configuration for Researcher updates + Research Program updates. @aislinnpearson here is a screenshot with some updated descriptions as well:

user-notifications

The other changes add the ability to un/subscribe to individual Proposal + Experiment entities individually, and notably, "receive a single alert for experiments that includes ... changes to all designs, plans, etc associated with this". So this aspect will be more related to a "Reporting dashboard" where we have discussed displaying changes in the past X Days/Weeks/Months as a separate feature, less specific to email notifications even. Does this sound OK @aislinnpearson ?

paul121 commented 4 months ago

Updated descriptions, this is done!

aislinnpearson commented 3 months ago

Hey @paul121,

So I wasn't going to bother mentioning this yesterday, but as @mstenta found an issue with the push I thought I'd see if it is also worth making this small change.

Basically I was going to ask if we could edit the text for the Research Program updates to it reads 'makes changes to a or any Research Program you are named on'. Most scientists have more than one, and to be fair this could be my error. It looks like the kind of spelling mistake I make.

The other one I am not sure about is the 'Email notifications' one. I feel like we could be a bit clearer there. Maybe one to discuss this evening.

aislinnpearson commented 3 months ago

Following on from the call please can we also edit Email Notifications so it says 'Switch off all e-mail notifications'. Ideally if we can link it with the UI features of switching off all the other buttons that would be amazing.

paul121 commented 3 months ago

Done!

Screenshot from 2024-06-04 09-52-48

aislinnpearson commented 3 months ago

Just tried this on our live instance. Looks great @paul121