Skip to content

[go_router] Documentation for StatefulShellRoute - #6308

Merged
auto-submit[bot] merged 5 commits into
flutter:mainfrom
tolo:statefulshellroute-docs
Aug 1, 2024
Merged

[go_router] Documentation for StatefulShellRoute#6308
auto-submit[bot] merged 5 commits into
flutter:mainfrom
tolo:statefulshellroute-docs

Conversation

@tolo

@tolotolo commented Mar 12, 2024

Copy link
Copy Markdown
Contributor

Adds documentation about StatefulShellRoute to the Configuration topic (configuration.md).

Fixesflutter/flutter#127209.

Pre-launch Checklist

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@tolo

tolo commented Mar 14, 2024

Copy link
Copy Markdown
ContributorAuthor

@johnpryan could you have a look at this?

@chunhtaichunhtai 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 for adding the document. Since we start to use code-excerpt, can you try to follow this when adding the example code in doc? https://github.com/flutter/flutter/wiki/Contributing-to-Plugins-and-Packages#add-or-converting-a-code-block

which is passed as the last argument to the builder function. Example:

```dart
StatefulShellRoute.indexedStack(

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.

@chunhtai

Copy link
Copy Markdown
Contributor

Hi @tolo are you still working on this pr?

@tolo

tolo commented Apr 29, 2024

Copy link
Copy Markdown
ContributorAuthor

Hi @tolo are you still working on this pr?

Yes, I'll have a look at code-excerpt and update the code / docs.

@chunhtai

Copy link
Copy Markdown
Contributor

closing this pr for now, feel free to reopen when you have time

@chunhtaichunhtai closed this Jun 6, 2024
@tolo

tolo commented Jun 6, 2024

Copy link
Copy Markdown
ContributorAuthor

@chunhtai I pushed new commits over a month ago which I don’t think you have reviewed yet (didn’t request a new review though, sorry).

@chunhtaichunhtai reopened this Jun 6, 2024
@chunhtai
chunhtai self-requested a review June 6, 2024 22:41

@chunhtaichunhtai 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.

LGTM

@chunhtai
chunhtai requested a review from hannah-hyjJuly 16, 2024 17:27

@hannah-hyjhannah-hyj left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, thank you for your contribution, can you rebase this to the head to resolve version conflicts?

@chunhtaichunhtai added the autosubmit Merge PR when tree becomes green via auto submit App label Aug 1, 2024
@auto-submit
auto-submitBot merged commit 9cc06ac into flutter:mainAug 1, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Aug 2, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Aug 2, 2024
auto-submitBot pushed a commit to flutter/flutter that referenced this pull request Aug 2, 2024
flutter/packages@27896d1...cc9ff47
2024-08-02 rexios@rexios.dev [tool] Add note about clang version used in CI to tool readme (flutter/packages#7273)
2024-08-02 ditman@gmail.com [ci] Re-enable google_maps_flutter_web integration tests. (flutter/packages#7269)
2024-08-01 ditman@gmail.com [web] Update package:web to ">=0.5.1 <2.0.0" (flutter/packages#7202)
2024-08-01 tobias@leafnode.se [go_router] Documentation for StatefulShellRoute (flutter/packages#6308)
2024-08-01 stuartmorgan@google.com [google_maps_flutter] Switch Android examples to TLHC (flutter/packages#7282)
2024-08-01 engine-flutter-autoroll@skia.org Manual roll Flutter from f817e51 to e2e4398 (5 revisions) (flutter/packages#7276)
2024-08-01 stuartmorgan@google.com [various] Clean up C++ formatting (flutter/packages#7272)
2024-08-01 tarrinneal@gmail.com [shared_preferences] fix stringlist bug (flutter/packages#7268)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages-flutter-autoroll
Please CC flutter-ecosystem@google.com,rmistry@google.com on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
TytaniumDev pushed a commit to TytaniumDev/flutter that referenced this pull request Aug 7, 2024
flutter/packages@27896d1...cc9ff47
2024-08-02 rexios@rexios.dev [tool] Add note about clang version used in CI to tool readme (flutter/packages#7273)
2024-08-02 ditman@gmail.com [ci] Re-enable google_maps_flutter_web integration tests. (flutter/packages#7269)
2024-08-01 ditman@gmail.com [web] Update package:web to ">=0.5.1 <2.0.0" (flutter/packages#7202)
2024-08-01 tobias@leafnode.se [go_router] Documentation for StatefulShellRoute (flutter/packages#6308)
2024-08-01 stuartmorgan@google.com [google_maps_flutter] Switch Android examples to TLHC (flutter/packages#7282)
2024-08-01 engine-flutter-autoroll@skia.org Manual roll Flutter from f817e51 to e2e4398 (5 revisions) (flutter/packages#7276)
2024-08-01 stuartmorgan@google.com [various] Clean up C++ formatting (flutter/packages#7272)
2024-08-01 tarrinneal@gmail.com [shared_preferences] fix stringlist bug (flutter/packages#7268)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages-flutter-autoroll
Please CC flutter-ecosystem@google.com,rmistry@google.com on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Buchimi pushed a commit to Buchimi/flutter that referenced this pull request Sep 2, 2024
flutter/packages@27896d1...cc9ff47
2024-08-02 rexios@rexios.dev [tool] Add note about clang version used in CI to tool readme (flutter/packages#7273)
2024-08-02 ditman@gmail.com [ci] Re-enable google_maps_flutter_web integration tests. (flutter/packages#7269)
2024-08-01 ditman@gmail.com [web] Update package:web to ">=0.5.1 <2.0.0" (flutter/packages#7202)
2024-08-01 tobias@leafnode.se [go_router] Documentation for StatefulShellRoute (flutter/packages#6308)
2024-08-01 stuartmorgan@google.com [google_maps_flutter] Switch Android examples to TLHC (flutter/packages#7282)
2024-08-01 engine-flutter-autoroll@skia.org Manual roll Flutter from f817e51 to e2e4398 (5 revisions) (flutter/packages#7276)
2024-08-01 stuartmorgan@google.com [various] Clean up C++ formatting (flutter/packages#7272)
2024-08-01 tarrinneal@gmail.com [shared_preferences] fix stringlist bug (flutter/packages#7268)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages-flutter-autoroll
Please CC flutter-ecosystem@google.com,rmistry@google.com on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
creatorpiyush pushed a commit to creatorpiyush/packages that referenced this pull request Jun 10, 2026
Adds documentation about StatefulShellRoute to the Configuration topic (configuration.md).
Fixesflutter/flutter#127209.
bisor0627 pushed a commit to bisor0627/packages that referenced this pull request Jun 19, 2026
Adds documentation about StatefulShellRoute to the Configuration topic (configuration.md).
Fixesflutter/flutter#127209.
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmitMerge PR when tree becomes green via auto submit Appp: go_router

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add docs for go_router StatefulShellRoute

3 participants

@tolo@chunhtai@hannah-hyj