open-sauced / landing-page

This is the new landing page for open sauced
https://opensauced.pizza
25 stars 28 forks source link

fix: now doc site rewrites work for all docs URLs #307

Closed nickytonline closed 1 month ago

nickytonline commented 1 month ago

Description

Fixes the rewrite rules for the docs site which now resides under https://opensauced.pizza/docs.

The issue was a missing trailing slash in the rewrite, and the additional rule for the root of the docs site to /docs was unnecessary.

Related Tickets & Documents

Fixes #300

Mobile & Desktop Screenshots/Recordings

Steps to QA

  1. Visit https://deploy-preview-307--opensauced-landing.netlify.app/docs. The docs site loads.
  2. Refresh the page, the docs site still loads.
  3. Navigate to https://deploy-preview-307--opensauced-landing.netlify.app/docs/maintainers/maintainers-guide-to-open-sauced/. The page loads.
  4. Refresh the https://deploy-preview-307--opensauced-landing.netlify.app/docs/maintainers/maintainers-guide-to-open-sauced/ page. The page loads.

Tier (staff will fill in)

[optional] What gif best describes this PR or how it makes you feel?

netlify[bot] commented 1 month ago

Deploy Preview for opensauced-landing ready!

Name Link
Latest commit 52f1eed9dc1037e176d0473b484ee2c03f7d0a17
Latest deploy log https://app.netlify.com/sites/opensauced-landing/deploys/6659e1abd76368000964918b
Deploy Preview https://deploy-preview-307--opensauced-landing.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

Lighthouse
1 paths audited
Performance: 100 (no change from production)
Accessibility: 91 (no change from production)
Best Practices: 92 (no change from production)
SEO: 83 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.