-
Notifications
You must be signed in to change notification settings - Fork 3.6k
[go_router] Update link to example for query and path parameters #3994
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
[go_router] Update link to example for query and path parameters #3994
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.
LGTM, thanks for fixing this!
Co-authored-by: chunhtai <[email protected]>
|
Could you update the PR to have a description, rather than just the placeholder about having a description? |
Updated ✅ |
|
@nilsreichardt can you fix the merge conflict? |
|
@chunhtai resolved ☑️ |
flutter/packages@e13b8c4...afe2f05 2023-06-08 [email protected] [image_picker] Add desktop support - platform interface (flutter/packages#4161) 2023-06-08 [email protected] [go_router] Update link to example for query and path parameters (flutter/packages#3994) 2023-06-08 49699333+dependabot[bot]@users.noreply.github.com [in_app_pur]: Bump org.jetbrains.kotlin:kotlin-bom from 1.8.0 to 1.8.21 in /packages/in_app_purchase/in_app_purchase_android/android (flutter/packages#3839) 2023-06-08 49699333+dependabot[bot]@users.noreply.github.com [image_picker]: Bump androidx.activity:activity from 1.7.0 to 1.7.1 in /packages/image_picker/image_picker_android/android (flutter/packages#3768) 2023-06-08 [email protected] [webview_flutter] Explicitly disable ATS in example apps (flutter/packages#4166) 2023-06-08 [email protected] Roll Flutter from 8a5c22e to 6e254a3 (12 revisions) (flutter/packages#4165) 2023-06-08 [email protected] Fix stale ignore: prefer_const_constructors from flutter/packages (flutter/packages#4154) 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://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
This PR updates the link to the example for query and path parameters because the old link doesn't exist anymore.
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.