backend icon indicating copy to clipboard operation
backend copied to clipboard

Published 20 hours ago • verified publisher icon source-academy

Fix Avenger Backlog Notifications

Open GabrielCWT opened this issue 1 year ago • 3 comments

  • Implement dedicated function for getting ungraded submissions. It fetches only the bare minimum (assessment_title, student_name, submission_id and course_id)
  • Refactor avenger backlog email template to use new function
  • Refactor cron job to use new function
  • Include status: submitted in test seed submissions
  • Refactor tests to use the dedicated function

Closes #1040

Closes #1032 Closes #939

GabrielCWT avatar Feb 24 '24 07:02 GabrielCWT

Coverage Status

coverage: 94.998% (+0.006%) from 94.992% when pulling 2095250c9a51643f24e134df8347007b5902ba3e on GabrielCWT:fix/notifications into 11154a6633b0cacac252bc01b35908f62d7e846d on source-academy:master.

coveralls avatar Feb 24 '24 07:02 coveralls

Changed back to draft as I would like to update notification links as well. @RichDom2185 any ideas on how would I get frontend url, is there an env variable I can use? Also I don't think there is an unsubscribe link as of yet

Edit: Probably can use cors_endpoints for frontend url.

GabrielCWT avatar Feb 25 '24 01:02 GabrielCWT

TODO:

  • Update unsubscribe link
  • Update url builder to use frontend url (Currently hardcoded to sourceacademy)

GabrielCWT avatar Feb 25 '24 01:02 GabrielCWT