VEuPathDB / EdaNewIssues

0 stars 0 forks source link

Ensure that all request emails sent to managers of protected studies contain links to the dashboard #704

Closed danicahelb closed 7 months ago

danicahelb commented 8 months ago

Some of the request emails sent to managers of protected studies are missing the link to the dashboard for that study.

For example, see Kalifabougou

image

Check the presenter file to ensure that all dashboards are linked

the html to add a link within text :

for passive approval:

To deny this request, visit the <a href="https://clinepidb.org/ce/app/study-access/$$DATASET_ID$$"> data access dashboard for the MAL-ED study </a> and change the approval status to "Denied" for this requestor.

for active approval:

To approve or deny this request, visit the <a href="https://clinepidb.org/ce/app/study-access/$$DATASET_ID$$"> data access dashboard for the MAL-ED study </a> and change the approval status to "Approved" or "Denied" for this requestor.

Or see JILINDE Costing survey in Kenya, as this study has the correct links:

image
danicahelb commented 8 months ago

related to https://github.com/VEuPathDB/EdaNewIssues/issues/662

nkittur-uga commented 8 months ago

I just reviewed the presenter carefully and all manager emails for protected studies do contain links to the dashboard. (It seems Kalifabougou may have been inadvertently left out but @danicahelb corrected that already?) This ticket could be closed.

danicahelb commented 8 months ago

outlook emails have a link, gmail does NOT have a link. for Kalifabougu, the link to the dashboard in outlook does NOT work...maybe this is why the link isn't getting rendered in gmail? @dmgaldi please check

danicahelb commented 8 months ago

there is a weird quote in the presenter, dan fixed this but it still isn't working

danicahelb commented 7 months ago

as of 11/15 this is still not working: link to passive tutorials is broken

image

this may be due to an extra space in the presenter

danicahelb commented 7 months ago

this now works