Skip to content

docs: add React Router framework guide and improve contributing guide#191

Open
ChloeGarciaMillerand wants to merge 2 commits into
vite-pwa:mainfrom
ChloeGarciaMillerand:docs/react-router-framework-guide
Open

docs: add React Router framework guide and improve contributing guide#191
ChloeGarciaMillerand wants to merge 2 commits into
vite-pwa:mainfrom
ChloeGarciaMillerand:docs/react-router-framework-guide

Conversation

@ChloeGarciaMillerand

Copy link
Copy Markdown

Add React Router Framework documentation and improve contributing guide

React Router Framework

As a React Router v7 user, I encountered some difficulties integrating Vite PWA, especially around the application entry point (root.tsx) and the preview setup, which did not work as expected for local testing.

I thought adding this documentation could help other developers facing similar issues.

Contributing Guide

I also spent some time understanding how to properly contribute to the existing documentation. This update aims to make the process clearer and help avoid similar confusion for future contributors.

Please feel free to let me know if any changes are needed in this PR. Thank you for your work!

@netlify

netlify Bot commented May 5, 2026

Copy link
Copy Markdown

Deploy Preview for vite-pwa-org ready!

Name Link
🔨 Latest commit 924b5e4
🔍 Latest deploy log https://app.netlify.com/projects/vite-pwa-org/deploys/69f9e1cfb1da110008ed8535
😎 Deploy Preview https://deploy-preview-191--vite-pwa-org.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

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

@userquin

userquin commented May 5, 2026

Copy link
Copy Markdown
Member

FYI: react router v7 not yet supported, remix did a huge refactor, next pwa plugin v2 will allow us to use vite env. Api and we will provide a new package for it.

Check this pr on my repo: userquin/vite-pwa-react-router#3

@ChloeGarciaMillerand

Copy link
Copy Markdown
Author

Thanks for the clarification.
What I added is currently working well in my React Router app, but my use case is fairly simple, so I may not have run into the limitations or unsupported cases yet.

I’ll keep following your work and future updates on this topic.

Thanks again for your feedback and for maintaining the project!

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.

2 participants