PIMS-1756: Template formatting and hyperlink fixes

[GrahamS-Quartech]

🎯 Summary

PIMS-1756:

• Made some slight optimizations and additions to the way relations are gathered for the project to generate a notification.
• There is now an additional field for the config object where it will get the title tag and uri for the templates. It uses FRONTEND_URL for the uri.
• Fixed templates using the wrong property to determine parcel or building for their if statements.
• Fixed some templates using the wrong property for ProjectNumber
• Fixed templates using the wrong property to determine NoteType
• The uri for the href of tags will now be dynamically constructed to link to the relevant project rather than just the homepage when appropriate
• Removed nullish coalescing operator from property accesses. It seems this is not supported by nunjucks and will fail even when accessing a non-null property.
• Fixed instances where the templates were expecting evaluations to use a Date object rather than a Year integer
• Fixed sort order of the statement that determines the assessment value to display so that it gets the most recent year rather than the oldest
• Corrected "Signin" to "Sign in"

...And probably a couple other things too.
Please give it a decent read over and let me know if I missed something.

🔰 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: 2554. 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 5534a2a 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 92.5%.

View more on Code Climate.

[dbarkowsky]

What's the best way to test this? Just send yourself emails, I suppose?

[GrahamS-Quartech]

What's the best way to test this? Just send yourself emails, I suppose?

Yeah, I would just choose a project with a few properties that have assessment values on them and try changing them into statuses that will prompt notifications. You can also add a new test project to ensure the new project notification works.

[TaylorFries]

notifications are sent for creation of projects and appropriate statuses changes (to: submitted, submitted exemption, approved for erp, denied, cancelled, in erp)
no issues detected.