Skip to content

docs: update service worker section in Web FAQ#13335

Merged
sfshaza2 merged 7 commits into
flutter:mainfrom
harshyadavDeveloper:fix/remove-pwa-strategy-faq
May 28, 2026
Merged

docs: update service worker section in Web FAQ#13335
sfshaza2 merged 7 commits into
flutter:mainfrom
harshyadavDeveloper:fix/remove-pwa-strategy-faq

Conversation

@harshyadavDeveloper

Copy link
Copy Markdown
Contributor

Description

Updates the Web FAQ to reflect the current state of service worker support.

Removes outdated references to the --pwa-strategy flag and clarifies that Flutter no longer generates or manages a service worker by default.

Issues fixed by this PR

Closes #13319

PRs or commits this PR depends on

None

Presubmit checklist

@harshyadavDeveloper harshyadavDeveloper requested review from a team and sfshaza2 as code owners April 30, 2026 10:15
@harshyadavDeveloper

Copy link
Copy Markdown
Contributor Author

Hi! I’ve updated the service worker section to remove outdated references to the --pwa-strategy flag and clarified the current behavior. Please let me know if any adjustments are needed.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request updates the Flutter web FAQ to clarify that service workers are no longer generated or managed by default, recommending manual configuration or third-party tools like Workbox for offline support. A review comment suggests improving the readability of a documentation link by using descriptive text instead of a technical identifier.

Comment thread sites/docs/src/content/platform-integration/web/faq.md Outdated
@flutter-website-bot

flutter-website-bot commented Apr 30, 2026

Copy link
Copy Markdown
Collaborator

Visit the preview URL for this PR (updated for commit 9647125):

https://flutter-docs-prod--pr13335-fix-remove-pwa-strategy-faq-zcs5sbmn.web.app

@harshyadavDeveloper

Copy link
Copy Markdown
Contributor Author

Thanks for pointing this out! I’ve removed the remaining ios-ffi reference usages to avoid broken links.

@parlough

parlough commented May 5, 2026

Copy link
Copy Markdown
Member

/gcbrun

@harshyadavDeveloper

Copy link
Copy Markdown
Contributor Author

Hello, this PR currently has merge conflicts blocking the merge. Could you please take a look and resolve them at your convenience?

@sfshaza2

Copy link
Copy Markdown
Contributor

/gcbrun

@kevmoo kevmoo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks so much!

@sfshaza2 sfshaza2 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks, @harshyadavDeveloper! Kevmoo says "great catch"!

@flutter-website-bot

Copy link
Copy Markdown
Collaborator

Staged preview of the updated docs.flutter.dev site (updated for commit 411f026):

https://flutter-docs-prod--docs-pr13335-fix-remove-pwa-strateg-ex0uftb8.web.app

@sfshaza2 sfshaza2 merged commit 8b2026b into flutter:main May 28, 2026
12 checks passed
@johnpryan

Copy link
Copy Markdown
Contributor

Thank you, @harshyadavDeveloper!

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.

Update Web FAQ so to mention the current state of service workers

6 participants