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

[go_router] docs: updated link in navigation.md to correct file path for push_with_shell_route.dart #6670

Merged
merged 2 commits into from
May 28, 2024

Conversation

altynbek132
Copy link
Contributor

@altynbek132 altynbek132 commented May 6, 2024

[go_router] docs: updated link in navigation.md to correct file path for push_with_shell_route.dart

Pre-launch Checklist

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

Copy link
Contributor

@chunhtai chunhtai left a 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?

@altynbek132 altynbek132 force-pushed the pr_url_fix branch 2 times, most recently from 84868a0 to 0505d5c Compare May 10, 2024 19:50
@altynbek132
Copy link
Contributor Author

Can you update version and change log so that this can be published to pub dev?

@chunhtai done

@Ankit-Kumar-Dhangar1
Copy link

i am using gorouter with bottomnavigation
quickaction is working fine for first time but when i change tab ,close it , and open it again using quickaction it is not opening to set page.

@altynbek132 altynbek132 requested a review from chunhtai May 11, 2024 09:55
Copy link
Contributor

@chunhtai chunhtai left a 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

@altynbek132
Copy link
Contributor Author

@chunhtai rebased

Copy link
Contributor

@hangyujin hangyujin left a comment

Choose a reason for hiding this comment

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

LGTM

@hangyujin
Copy link
Contributor

Hi, can you rebase again and you also update the version number in pubspec.yaml

@altynbek132
Copy link
Contributor Author

@hangyujin @chunhtai done

Copy link
Contributor

@chunhtai chunhtai left a comment

Choose a reason for hiding this comment

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

LGTM

@chunhtai chunhtai added the autosubmit Merge PR when tree becomes green via auto submit App label May 28, 2024
@auto-submit auto-submit bot merged commit b4d134e into flutter:main May 28, 2024
74 checks passed
ThangVuNguyenViet pushed a commit to ThangVuNguyenViet/packages that referenced this pull request May 29, 2024
…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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
autosubmit Merge PR when tree becomes green via auto submit App p: go_router
Projects
None yet
4 participants