The moves the old workflow notifications into status change requests. There is more work to be done with notifications in other issues/PRs. Hard code DDs were removed so that it now just notifies the DD and Deputy DD to review based on the owning division.
Issue
2316
2465
fixes: #2134
see also #1471 for future work
How to test
run the tests
Or run the app and request a status change and approve it and view the notifications
Definition of Done Checklist
[x] OESA: Code refactored for clarity
[x] OESA: Dependency rules followed
[x] Automated unit tests updated and passed
[x] Automated integration tests updated and passed
What changed
The moves the old workflow notifications into status change requests. There is more work to be done with notifications in other issues/PRs. Hard code DDs were removed so that it now just notifies the DD and Deputy DD to review based on the owning division.
Issue
2316
2465
fixes: #2134 see also #1471 for future work
How to test
run the tests Or run the app and request a status change and approve it and view the notifications
Definition of Done Checklist