PIMS-2132: Fields included in the notification emails

[LawrenceLau2020]

🎯 Summary

PIMS-2132

Changes

• Added the property's location ("Site Location") into the email template notifications for projects that have been either "Submitted" or "Approved for ERP".

Testing

• Need to test 2 different project statuses which use these templates: "Submitted & "Approved for ERP".
• If PIMS-1946 Cancel/Stop Notifications #2726 hasn't been merged to dev yet, open any project that has already been "submitted" and change the status back to "Submitted" or back to "Approved for ERP" to trigger the email notifications.
  • You should be able to see the new line displaying the property's City, located under "Site Address" in the email notification.
• You could also create a new project and submit it.
• Afterwards, you can also change the status to "Approved for ERP" and check that the "Site Location" is included in the email notifications.
• Testing the rollback can be optional, but might be a good idea to see that it works.
  • For this you can either review the body column of the notifications table in the database, and/or trigger the above notifications again.

🔰 Checklist

• I have read and agree with the following checklist and am following the guidelines in our Code of Conduct document.

• I have performed a self-review of my code.
• I have commented my code, particularly in hard-to-understand areas.
• I have made corresponding changes to the documentation where required.
• I have tested my changes to the best of my ability.
• My changes generate no new warnings.

[github-actions]

🚀 Deployment Information

The Express API Image has been built with the tag: 2727. Please make sure to utilize this specific tag when promoting these changes to the TEST and PROD environments during the API deployment. For more updates please monitor Image Tags Page on Wiki.

[codeclimate]

Code Climate has analyzed commit 0d25145 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 93.4%.

View more on Code Climate.

[LawrenceLau2020]

There didn't seem to be a way to do a global search and replace based on the differences in formatting in template id=1, so had to split it up into 2 separate queries.

[dbarkowsky]

Tested in both directions. Migrates and reverts successfully.
The Site Location line is nicely formatted in all cases.
Thank you for updating the nunjucks templates too.