equinor / energyvision

Home of the equinor.com website
https://www.equinor.com
MIT License
34 stars 8 forks source link

Move redirect and external redirect out of middleware #2115

Open fernandolucchesi opened 9 months ago

fernandolucchesi commented 9 months ago

Potential fix is fetching redirects during build time

BorghildSelle commented 8 months ago

Todo:

BorghildSelle commented 8 months ago

@padms @millianapia Can you test redirects in staging environment? Needs to be checked:

meols commented 8 months ago

@BorghildSelle - nothing for me to test here I assume? :smile:

BorghildSelle commented 8 months ago

@meols not yet, developers must test first :)

meols commented 2 months ago

@BorghildSelle - will this fix issues in the siteimprove reports linked below? We see a spike around 25 April in several of the reports, and don't understand why why we have a sudden increase. https://my2.siteimprove.com/SEOv2/550893/Recommendations/RedirectsChain https://my2.siteimprove.com/SEOv2/550893/Recommendations/Redirects301 https://my2.siteimprove.com/SEOv2/550893/Recommendations/RedirectsNot301 https://my2.siteimprove.com/SEOv2/550893/Recommendations/RedirectsMixed

BorghildSelle commented 2 weeks ago

@padms Put this back in to do, so we can do the suggestion we talked about with returning after all conditions have been checked