Fix Handling of "/contact-us" Within App and Emails

[aaronskiba]

Related issue:

• https://github.com/portagenetwork/roadmap/issues/385

Please complete the following fields as applicable:

What version of the DMPRoadmap code are you running? (e.g. v2.2.0)

• current production version (3.3.1+portage-3.2.2)
• 4.0.2+portage-4.0.0-alpha-1

Explanation -Emails to users state please contact us at or visit https://dmp-pgd.ca/contact-us. However, the Contact Us button within the app links to https://portagenetwork.ca/contact-us/ (which re-routes to https://alliancecan.ca/en/services/research-data-management).

Received Email (Testing on UAT)

dmp-pgd.ca/contact-us

Contact Us Link Within App

[lagoan]

@aaronskiba Can you update the email to reflect our application contact us web page?

[aaronskiba]

There is quite a bit of /contact-us handling throughout the app: 1) contact_us_url, contact_us_path, Rails.application.routes.url_helpers.contact_us_url, andRails.application.routes.url_helpers.contact_us_path, and are assigned via the installed contact_us gem 2) contact_us_path is overridden in application_helper.rb via def contact_us_path 3) _dmproadmap.rb has a (commented-out) variable for setting config.x.organisation.contact_us_url 4) branding.yml has a (commented-out) variable for setting Rails.application.config.branding[:organisation][:contact_us_url]

The overriding via 2. does not change Rails.application.routes.url_helpers.contact_us_url or Rails.application.routes.url_helpers.contact_us_url. I'm thinking that the references of Rails.application.routes.url_helpers.contact_us_url should be changed to contact_us_url and the instances of Rails.application.routes.url_helpers.contact_us_path should be changed to contact_us_path.

[aaronskiba]

Here is one other spot where "Contact Us" will have to be updated: /org/admin/:id/admin_edit