Skip to content

docs: change _rewrites to _redirects in docs #1895

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

Merged

Conversation

scouttyg
Copy link

Summary

The current Netlify docs mention a _rewrites file, but I don't believe such a file actually is supported -- the correct file I believe is _redirects

Test plan

  1. Visit the Deploy Preview (insert link to specific page) ...

Relevant links (GitHub issues, Notion docs, etc.)

@scouttyg scouttyg requested a review from a team January 19, 2023 15:03
@netlify
Copy link

netlify bot commented Jan 19, 2023

Deploy Preview for netlify-plugin-nextjs-demo ready!

Name Link
🔨 Latest commit 67d1b26
🔍 Latest deploy log https://app.netlify.com/sites/netlify-plugin-nextjs-demo/deploys/63c95bc7f6c72c0009aced2c
😎 Deploy Preview https://deploy-preview-1895--netlify-plugin-nextjs-demo.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@netlify
Copy link

netlify bot commented Jan 19, 2023

👷 Deploy request for netlify-plugin-nextjs-static-root-demo pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 67d1b26

@netlify
Copy link

netlify bot commented Jan 19, 2023

👷 Deploy request for next-plugin-edge-middleware pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 67d1b26

@netlify
Copy link

netlify bot commented Jan 19, 2023

👷 Deploy request for next-i18next-demo pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 67d1b26

@netlify
Copy link

netlify bot commented Jan 19, 2023

👷 Deploy request for netlify-plugin-nextjs-nx-monorepo-demo pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 67d1b26

@netlify
Copy link

netlify bot commented Jan 19, 2023

👷 Deploy request for netlify-plugin-nextjs-export-demo pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 67d1b26

@netlify
Copy link

netlify bot commented Jan 19, 2023

👷 Deploy request for next-plugin-canary pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 67d1b26

@netlify
Copy link

netlify bot commented Jan 19, 2023

👷 Deploy request for netlify-plugin-nextjs-next-auth-demo pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 67d1b26

@scouttyg scouttyg changed the title fix: change _rewrites to _redirects in docs docs: change _rewrites to _redirects in docs Jan 19, 2023
Copy link
Contributor

@ascorbic ascorbic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Good catch

@kodiakhq kodiakhq bot merged commit b046ef5 into opennextjs:main Jan 19, 2023
@scouttyg scouttyg deleted the change-rewrites-to-redirects-in-docs branch January 19, 2023 19:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants