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_builder] Add test for onExit #6614

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

ValentinVignal
Copy link
Contributor

Follows #6495
Fixes flutter/flutter#137394

Pre-launch Checklist

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

@ValentinVignal
Copy link
Contributor Author

The tests are failing because of #6623

…dd-tests-for-on-exit

# Conflicts:
#	packages/go_router_builder/CHANGELOG.md
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 requested a review from hangyujin May 9, 2024 21:39
auto-submit bot pushed a commit that referenced this pull request May 14, 2024
…o the `onExit` (#6623)

While working on #6614, I notice some issues with the state that is given to the `onExit` (see the run https://github.com/flutter/packages/pull/6614/checks?check_run_id=24284539541)

This PR uses the correct configuration to build the state and pass it to the `onExit`
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

@ValentinVignal ValentinVignal force-pushed the go-router-builder/Add-tests-for-on-exit branch from 39d7939 to 0f8be0e Compare May 14, 2024 07:35
…dd-tests-for-on-exit

# Conflicts:
#	packages/go_router_builder/CHANGELOG.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants