vite-pwa / docs

Documentation: PWA integrations for Vite and the ecosystem
https://vite-pwa-org.netlify.app/
MIT License
197 stars 44 forks source link

feat: add integrations and PHP (Laravel) docs #23

Open userquin opened 1 year ago

netlify[bot] commented 1 year ago

Deploy Preview for vite-pwa-org ready!

Name Link
Latest commit efb3b8f5cbdfb68c07fdd8c0c4097b9f598d980f
Latest deploy log https://app.netlify.com/sites/vite-pwa-org/deploys/63e3d840b8ec950008db6e19
Deploy Preview https://deploy-preview-23--vite-pwa-org.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.

anburocky3 commented 1 year ago

@userquin Can i provide the example VitePWA() config in the bottom of the docs?

anburocky3 commented 1 year ago

@userquin Regarding https://deploy-preview-23--vite-pwa-org.netlify.app/integrations/laravel.html#registering-service-worker

The service worker is injecting using this option and looks like we don't need to have it in app.ts to inject it using registerSW()

injectRegister: 'script',
anburocky3 commented 1 year ago

@userquin I have made some changes, I couldn't push my commit.

image

userquin commented 1 year ago

you can add what you want, I just create the minimal to allow you start the docs without configuring VitePress: added some entries to the docs, feel free to change it

anburocky3 commented 1 year ago

you can add what you want, I just create the minimal to allow you start the docs without confiring VitePress: added some entries to the docs, feel free to change it

Thank you, i changed few. But i couldn't push to this branch. Permission denied.

userquin commented 1 year ago

Permission denied.

Check my comment about this in the issue.

anburocky3 commented 1 year ago

As you said, gave a PR here #24

slavarazum commented 1 year ago

Is it compatible with InertiaJS and Livewire, the two basic Laravel stacks?