feature/DEVSU-2656 email notification

[sshugsc]

DEVSU-2656:

• Add notification endpoints to swagger
• Use query params for notification GET
• Only user self and admin can POST/DELETE notifications

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 68.42105% with 6 lines in your changes missing coverage. Please review.

Project coverage is 77.07%. Comparing base (5ea6d17) to head (baba63b).

Files with missing lines	Patch %	Lines
app/libs/email.js	0.00%	0 Missing and 3 partials ⚠️
app/routes/notification/notification.js	70.00%	3 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop     #447      +/-   ##
===========================================
+ Coverage    77.04%   77.07%   +0.02%     
===========================================
  Files          183      183              
  Lines         6395     6399       +4     
  Branches       758      761       +3     
===========================================
+ Hits          4927     4932       +5     
+ Misses        1382     1379       -3     
- Partials        86       88       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[elewis2]

I think this means a new record won't be created each time a notification is required - a new one is needed each time there's a new task because this table lets us track whether the queue is working. We should replace this with eg Arena for monitoring bullmq tasks - and clear out the backlog in this table - but in the meantime using findOrCreate here instead of create means the record is not useful.

[sshugsc]

agree. it should use create instead of findOrCreate.