Skip to content
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

Update wasm docs to be clear that additional headers are now optional #11706

Merged
merged 1 commit into from
Feb 11, 2025

Conversation

kevmoo
Copy link
Contributor

@kevmoo kevmoo commented Feb 11, 2025

Fixes #11649

@kevmoo kevmoo requested review from sfshaza2, antfitch, parlough and a team as code owners February 11, 2025 01:05
@kevmoo
Copy link
Contributor Author

kevmoo commented Feb 11, 2025

FYI: I skipped details about release versions. This is the NEW behavior as of 3.29

CC @yjbanov

@kevmoo
Copy link
Contributor Author

kevmoo commented Feb 11, 2025

CC @eyebrowsoffire

@flutter-website-bot
Copy link
Collaborator

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

https://flutter-docs-prod--pr11706-i11649-wasm-header-update-cac97nl9.web.app

@kevmoo
Copy link
Contributor Author

kevmoo commented Feb 11, 2025

Copy link
Contributor

@sfshaza2 sfshaza2 left a comment

Choose a reason for hiding this comment

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

lgtm

@sfshaza2 sfshaza2 merged commit 7e2d11b into main Feb 11, 2025
9 checks passed
@sfshaza2 sfshaza2 deleted the i11649_wasm_header_update branch February 11, 2025 15:09
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.

web/wasm: note that changing HTTP headers is now only recommended, not required for Wasm
3 participants