Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix missing redirects in documentation URLs #126

Merged
merged 1 commit into from
Oct 3, 2023

Conversation

rainerdema
Copy link
Contributor

@rainerdema rainerdema commented Oct 3, 2023

Summary

  1. Customization Guide Redirect

    • From: /developers/customizations/overview
    • To: /customization/customizing-your-storefront
  2. Installation Guide Redirect

    • From: /developers/getting-started/first-time-installation
    • To: /getting-started/installing-solidus

ℹ️ More details here: solidusio/solidus#5404 (comment)

Checklist

  • I have followed the Diátaxis framework in my PR.
  • I have verified that the preview environment works correctly.

Test redirect

  1. Customization Guide Redirect

  2. Installation Guide Redirect

Update the redirects within the Docusaurus configuration to resolve 
issues with outdated or incorrect URLs in the Solidus guides.

- Update the link redirecting from the older customization guide path 
  to the newer one.
- Fix the link which was leading to a 404, directing it to the proper 
  installation guide.
@rainerdema rainerdema self-assigned this Oct 3, 2023
@cloudflare-pages
Copy link

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: b075fcb
Status: ✅  Deploy successful!
Preview URL: https://84c919d8.edgeguides.pages.dev
Branch Preview URL: https://rainerd-fix-redirects.edgeguides.pages.dev

View logs

@rainerdema rainerdema merged commit 2790a23 into main Oct 3, 2023
1 check passed
@rainerdema rainerdema deleted the rainerd/fix-redirects branch October 3, 2023 13:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants