carpentries / amy

A web-based workshop administration application built using Django.
https://amy.carpentries.org
MIT License
113 stars 72 forks source link

Scheduled email view should include email-specific information #2638

Closed maneesha closed 5 months ago

maneesha commented 5 months ago

In the current scheduled email view (list view and sample item view we do not easily see details about what email that is, especially as the template variables do not render.

@pbanaszkiewicz Can you advise what options we have so we can tell exactly what email that is? Depending on the email, this may mean displaying the event slug, person name, award, etc. One option may be for the scheduled email list, to include the related object in that table. The ID currently displayed in the scheduled emails table view is not relevant to the end user so that space could be used for this instead. Then for the item view, to move the related object (currently at the bottom of the view) to the top to make it more prominent.

pbanaszkiewicz commented 5 months ago

@maneesha

Proposal for the scheduled email list:

image

Proposal for the details of scheduled email:

image

What do you think?

I made sure that all currently related objects can be linked (Instructor Recruitment Signup leads to Instructor Recruitment details, Award leads to Person details, and Event, Person and Membership lead to their detail pages).

maneesha commented 5 months ago

The list view looks good. The detail view is also good - with a question about the id. Is there a reason the human user would ever need to see this? If not, I would hide it completely. If it can be useful (for example in troubleshooting) then I would move it to the bottom.

pbanaszkiewicz commented 5 months ago

@maneesha I'm going to move "ID" row in scheduled email details to the bottom of that table.

In regards to the notifications I mentioned during our meeting today. After merging a person this is the notification message we receive:

image

I'm going to change it to:

image