Skip to content

docs: Add documentation for Laravel framework #182

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

sfreytag
Copy link

@sfreytag sfreytag commented May 22, 2025

This PR results from vite-pwa/vite-plugin-pwa#431, specifically this comment.

Base on the sfreytag/laravel-vite-pwa repository
Copy link

netlify bot commented May 22, 2025

Deploy Preview for vite-pwa-org ready!

Name Link
🔨 Latest commit f8d46c9
🔍 Latest deploy log https://app.netlify.com/projects/vite-pwa-org/deploys/682ef27ad289160008621004
😎 Deploy Preview https://deploy-preview-182--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 project configuration.

@sfreytag sfreytag changed the title Add documentation for Laravel framework docs: Add documentation for Laravel framework May 22, 2025
@francoism90
Copy link

I would like to have customize this myself, instead of cloning a repo that has a prebuild demo.

@sfreytag
Copy link
Author

I would like to have customize this myself, instead of cloning a repo that has a prebuild demo.

With reference to the setup section I propose doing this by either the diff to vanilla laravel or following the commit history.

As a documentation technique there's a precedent for this in eg the Vue2 to 3 migration guide. I quite like it as a formal way of describing what is required. Written documentation would be a human version of that diff, essentially, which is arguably less precise.

Does that work for you?

@userquin
Copy link
Member

userquin commented Jun 5, 2025

will review the PR next weekend, looks good to me

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.

3 participants