Skip to content

Conversation

sm-sayedi
Copy link
Collaborator

And update Flutter's supporting libraries to match.

With this upgrade, we can now use the route transition duration in tests to wait the minimum amount of time needed for the route animation to complete.

Related upstream PR: flutter/flutter#171109

@sm-sayedi sm-sayedi added the integration review Added by maintainers when PR may be ready for integration label Jul 23, 2025
@sm-sayedi sm-sayedi self-assigned this Jul 23, 2025
@sm-sayedi sm-sayedi requested a review from gnprice July 23, 2025 20:31
@sm-sayedi sm-sayedi assigned gnprice and unassigned sm-sayedi Jul 23, 2025
@sm-sayedi sm-sayedi force-pushed the flutter-upgrade-25/07/24 branch 2 times, most recently from cf27c72 to ae988d7 Compare July 23, 2025 21:35
@gnprice gnprice force-pushed the flutter-upgrade-25/07/24 branch from ae988d7 to fa35ed4 Compare July 24, 2025 19:14
@gnprice
Copy link
Member

gnprice commented Jul 24, 2025

Thanks! Looks good; merging.

And update Flutter's supporting libraries to match.
@gnprice gnprice force-pushed the flutter-upgrade-25/07/24 branch from fa35ed4 to 55d055f Compare July 24, 2025 20:45
@gnprice gnprice merged commit 55d055f into zulip:main Jul 24, 2025
1 check passed
gnprice added a commit to gnprice/zulip-flutter that referenced this pull request Jul 25, 2025
This reverts commit 55d055f (zulip#1738).

That change was described as upgrading Flutter and its supporting
libraries, but it looks like it accidentally included other upgrades,
something like `flutter pub upgrade`.  It also didn't include the
CocoaPods updates for those upgrades; see zulip#1746.

So revert that change, to get back to a well-understood state.
Then we can retry the upgrade.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
integration review Added by maintainers when PR may be ready for integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants