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 page for beta and new docs #10249

Merged
merged 16 commits into from
Mar 13, 2024
Merged

Update wasm page for beta and new docs #10249

merged 16 commits into from
Mar 13, 2024

Conversation

MaryaBelanger
Copy link
Contributor

@MaryaBelanger MaryaBelanger commented Mar 6, 2024

Fixes #10159 --- staged page

Updated the page to:

  1. Point to new package:web and JS interop docs
  2. Point to the beta channel instead of master

Things that are NOT in this update (correct me if I'm wrong?):

  1. Dual compile

    • ref: Wasm/JS Dual Compile with the flutter tool flutter#141396
    • Thought about adding a "Coming soon" section at the end of the page, (or as a subsection to the Serve section) describing what the feature will look like at a conceptual level -- thoughts?
    • Will need to update the Run and Serve sections once the --wasm output goes to build/web not build/web_wasm
  2. is:wasm-ready tag on pub.dev

@flutter-website-bot
Copy link
Collaborator

flutter-website-bot commented Mar 6, 2024

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

https://flutter-docs-prod--pr10249-wasm-update-1wumv5nf.web.app

@MaryaBelanger MaryaBelanger marked this pull request as ready for review March 6, 2024 23:36
@MaryaBelanger MaryaBelanger added the review.tech Awaiting Technical Review label Mar 6, 2024
src/platform-integration/web/wasm.md Outdated Show resolved Hide resolved
@MaryaBelanger MaryaBelanger self-assigned this Mar 7, 2024
@MaryaBelanger
Copy link
Contributor Author

@kevmoo I set "last updated" to today (March 7), but I switched to the beta channel and followed the steps the page describes in this section and didn't see the "Experimental options" section at the end of the flutter build web --help output like it says I should.

So I'm guessing today is not actually the day; should I change "Last updated" to tomorrow?

@kevmoo
Copy link
Contributor

kevmoo commented Mar 7, 2024

@MaryaBelanger – as of RIGHT NOW beta has not landed. Hoping it's soon.

src/platform-integration/web/wasm.md Show resolved Hide resolved
src/platform-integration/web/wasm.md Outdated Show resolved Hide resolved
src/platform-integration/web/wasm.md Outdated Show resolved Hide resolved
src/platform-integration/web/wasm.md Show resolved Hide resolved
Copy link
Contributor

@kevmoo kevmoo left a comment

Choose a reason for hiding this comment

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

one thing to delete. then we need to deal with the {{}} bits

Copy link
Contributor

@kevmoo kevmoo left a comment

Choose a reason for hiding this comment

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

One nit then awesome!

Co-authored-by: Kevin Moore <kevmoo@users.noreply.github.com>
Co-authored-by: Kevin Moore <kevmoo@users.noreply.github.com>
@@ -3,45 +3,53 @@ title: Support for WebAssembly (Wasm)
description: >-
Current status of Flutter's experimental support for WebAssembly (Wasm).
short-title: Wasm
last-update: November 21, 2023
last-update: March 8, 2024
Copy link
Contributor

Choose a reason for hiding this comment

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

Just tagging this line as requires update.

Copy link
Member

@parlough parlough left a comment

Choose a reason for hiding this comment

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

Thanks for updating this @MaryaBelanger!

A few suggestions to consider:

src/platform-integration/web/wasm.md Outdated Show resolved Hide resolved
src/platform-integration/web/wasm.md Outdated Show resolved Hide resolved
src/platform-integration/web/wasm.md Outdated Show resolved Hide resolved
src/platform-integration/web/wasm.md Outdated Show resolved Hide resolved
src/platform-integration/web/wasm.md Outdated Show resolved Hide resolved
src/platform-integration/web/wasm.md Outdated Show resolved Hide resolved
src/platform-integration/web/wasm.md Outdated Show resolved Hide resolved
src/platform-integration/web/wasm.md Outdated Show resolved Hide resolved
src/platform-integration/web/wasm.md Outdated Show resolved Hide resolved
src/platform-integration/web/wasm.md Outdated Show resolved Hide resolved
MaryaBelanger and others added 2 commits March 11, 2024 15:10
Co-authored-by: Parker Lougheed <parlough@gmail.com>
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.

A few tweaks.

src/platform-integration/web/wasm.md Outdated Show resolved Hide resolved
src/platform-integration/web/wasm.md Outdated Show resolved Hide resolved
src/platform-integration/web/wasm.md Outdated Show resolved Hide resolved
src/platform-integration/web/wasm.md Outdated Show resolved Hide resolved
src/platform-integration/web/wasm.md Outdated Show resolved Hide resolved
MaryaBelanger and others added 3 commits March 12, 2024 10:23
Co-authored-by: Shams Zakhour (ignore Sfshaza) <44418985+sfshaza2@users.noreply.github.com>
@sfshaza2 sfshaza2 added the review.await-update Awaiting Updates after Edits label Mar 12, 2024
@kevmoo
Copy link
Contributor

kevmoo commented Mar 13, 2024

can we land this...like...now? 😄

@MaryaBelanger MaryaBelanger merged commit 40e0d1c into main Mar 13, 2024
10 checks passed
@MaryaBelanger MaryaBelanger deleted the wasm-update branch March 13, 2024 15:32
@sfshaza2 sfshaza2 removed review.tech Awaiting Technical Review review.await-update Awaiting Updates after Edits labels Mar 14, 2024
atsansone pushed a commit to atsansone/website that referenced this pull request Mar 18, 2024
Fixes flutter#10159 

---------

Co-authored-by: Kevin Moore <kevmoo@users.noreply.github.com>
Co-authored-by: Parker Lougheed <parlough@gmail.com>
Co-authored-by: Shams Zakhour (ignore Sfshaza) <44418985+sfshaza2@users.noreply.github.com>
atsansone pushed a commit to atsansone/website that referenced this pull request Apr 5, 2024
Fixes flutter#10159 

---------

Co-authored-by: Kevin Moore <kevmoo@users.noreply.github.com>
Co-authored-by: Parker Lougheed <parlough@gmail.com>
Co-authored-by: Shams Zakhour (ignore Sfshaza) <44418985+sfshaza2@users.noreply.github.com>
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 WASM page with info from Feb 2024 release
8 participants