Skip to content

[compass_app] Refresh home after booking - #2880

Open
Ersaoktaviannn wants to merge 1 commit into
flutter:mainfrom
Ersaoktaviannn:fix/compass-home-refresh
Open

[compass_app] Refresh home after booking#2880
Ersaoktaviannn wants to merge 1 commit into
flutter:mainfrom
Ersaoktaviannn:fix/compass-home-refresh

Conversation

@Ersaoktaviannn

@Ersaoktaviannn Ersaoktaviannn commented Aug 13, 2026

Copy link
Copy Markdown

Summary

  • reload HomeViewModel when the retained home route becomes active again
  • keep route-listener ownership alongside the view model in HomeScreenContainer
  • add a regression test covering a booking created while a child route is active
  • document the fix in the changelog

Fixes #2877.

Testing

  • flutter analyze --no-pub (from compass_app/app)
  • flutter test --no-pub (68 tests)
  • verified the regression test fails without the route listener and passes with the fix

Pre-launch Checklist

  • I read the Flutter Style Guide recently, and have followed its advice.
  • I signed the CLA.
  • I read the Contributors Guide.
  • I have added sample code updates to the changelog.
  • I updated/added relevant documentation (no public API documentation changes are needed).

@google-cla

google-cla Bot commented Aug 13, 2026

Copy link
Copy Markdown

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@Ersaoktaviannn

Copy link
Copy Markdown
Author

@googlebot rescan

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[compass_app] UI state never updates for the HomeScreen (regression after the implementation of #2788)

1 participant