-
Notifications
You must be signed in to change notification settings - Fork 3.6k
[go_router] docs: updated link in navigation.md to correct file path for push_with_shell_route.dart #6670
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
Conversation
chunhtai
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you update version and change log so that this can be published to pub dev?
84868a0 to
0505d5c
Compare
@chunhtai done |
|
i am using gorouter with bottomnavigation |
chunhtai
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, can you rebase again? this package has a lot of traffic
|
@chunhtai rebased |
hannah-hyj
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
Hi, can you rebase again and you also update the version number in pubspec.yaml |
…for push_with_shell_route.dart
…h fix for URL in `navigation.md`
|
@Hangyujin @chunhtai done |
chunhtai
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
…for push_with_shell_route.dart (flutter#6670) [go_router] docs: updated link in navigation.md to correct file path for push_with_shell_route.dart
…le path for push_with_shell_route.dart (flutter/packages#6670)
flutter/packages@a933c30...31d3329 2024-05-28 49699333+dependabot[bot]@users.noreply.github.com [pigeon]: Bump org.jetbrains.kotlin:kotlin-gradle-plugin from 1.9.22 to 2.0.0 in /packages/pigeon/platform_tests/test_plugin/android (flutter/packages#6815) 2024-05-28 [email protected] [google_maps_flutter] Implement polyline patterns in google maps ios (flutter/packages#5757) 2024-05-28 [email protected] [many] Remove references to v1 embedding (flutter/packages#6494) 2024-05-28 [email protected] [go_router] docs: updated link in navigation.md to correct file path for push_with_shell_route.dart (flutter/packages#6670) 2024-05-28 49699333+dependabot[bot]@users.noreply.github.com [webview]: Bump androidx.webkit:webkit from 1.10.0 to 1.11.0 in /packages/webview_flutter/webview_flutter_android/android (flutter/packages#6805) 2024-05-28 [email protected] Roll Flutter from 0b31ffc to a1a33e6 (6 revisions) (flutter/packages#6822) 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 [email protected],[email protected] 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
flutter/packages@a933c30...31d3329 2024-05-28 49699333+dependabot[bot]@users.noreply.github.com [pigeon]: Bump org.jetbrains.kotlin:kotlin-gradle-plugin from 1.9.22 to 2.0.0 in /packages/pigeon/platform_tests/test_plugin/android (flutter/packages#6815) 2024-05-28 [email protected] [google_maps_flutter] Implement polyline patterns in google maps ios (flutter/packages#5757) 2024-05-28 [email protected] [many] Remove references to v1 embedding (flutter/packages#6494) 2024-05-28 [email protected] [go_router] docs: updated link in navigation.md to correct file path for push_with_shell_route.dart (flutter/packages#6670) 2024-05-28 49699333+dependabot[bot]@users.noreply.github.com [webview]: Bump androidx.webkit:webkit from 1.10.0 to 1.11.0 in /packages/webview_flutter/webview_flutter_android/android (flutter/packages#6805) 2024-05-28 [email protected] Roll Flutter from 0b31ffc to a1a33e6 (6 revisions) (flutter/packages#6822) 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 [email protected],[email protected] 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
[go_router] docs: updated link in navigation.md to correct file path for push_with_shell_route.dart
Pre-launch Checklist
dart format.)[shared_preferences]pubspec.yamlwith an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.CHANGELOG.mdto add a description of the change, following repository CHANGELOG style.///).If you need help, consider asking for advice on the #hackers-new channel on Discord.