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

[CP] Fix for CupertinoPicker and ListWheelViewport crash #126782

Closed
nt4f04uNd opened this issue May 14, 2023 · 8 comments
Closed

[CP] Fix for CupertinoPicker and ListWheelViewport crash #126782

nt4f04uNd opened this issue May 14, 2023 · 8 comments
Assignees
Labels
cp: approved Approved cherry-pick request cp: merge-to-stable Cherry-picks that should be merged to stable cp: review Cherry-picks in the review queue

Comments

@nt4f04uNd
Copy link
Member

nt4f04uNd commented May 14, 2023

issue_link

#126491

Commit Hash

ae82f3c

Target

stable

pr_link

#126804

Impacted Users

Flutter devs who use CupertinoPicker or ListWheelViewport with certain configurations

Impact Description

It impacts only development builds, will not cause any problems in release builds.

Workaround

N/A

Risk

low

Test Coverage

yes

Validation Steps

No response

@CaseyHillers
Copy link
Contributor

@Piinks can you review this CP request? If it looks good to you, add the "cp: approved" label.

@CaseyHillers CaseyHillers added the cp: merge-to-stable Cherry-picks that should be merged to stable label May 15, 2023
@CaseyHillers
Copy link
Contributor

@nt4f04uNd for the PR link section, can you open a new PR that CP's this to the current stable branch? The current stable is on flutter-3.10-candidate.1

@nt4f04uNd
Copy link
Member Author

nt4f04uNd commented May 15, 2023

@nt4f04uNd for the PR link section, can you open a new PR that CP's this to the current stable branch? The current stable is on flutter-3.10-candidate.1

Ahh, got you, done.

It wasn't clear from the template, to be honest. Where can one look up which release branch they should open their PR against beforehand?
I now see that it was very clear, I just didn't read it 😅

@Piinks
Copy link
Contributor

Piinks commented May 15, 2023

CP LGTM

@Piinks Piinks added the cp: approved Approved cherry-pick request label May 15, 2023
XilaiZhang pushed a commit that referenced this issue May 15, 2023
…26804)

Fixes #126782

## Pre-launch Checklist

- [x] I read the [Contributor Guide] and followed the process outlined
there for submitting PRs.
- [x] I read the [Tree Hygiene] wiki page, which explains my
responsibilities.
- [x] I read and followed the [Flutter Style Guide], including [Features
we expect every widget to implement].
- [x] I signed the [CLA].
- [x] I listed at least one issue that this PR fixes in the description
above.
- [x] I updated/added relevant documentation (doc comments with `///`).
- [x] I added new tests to check the change I am making, or this PR is
[test-exempt].
- [x] All existing and new tests are passing.

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

<!-- Links -->
[Contributor Guide]:
https://github.com/flutter/flutter/wiki/Tree-hygiene#overview
[Tree Hygiene]: https://github.com/flutter/flutter/wiki/Tree-hygiene
[test-exempt]:
https://github.com/flutter/flutter/wiki/Tree-hygiene#tests
[Flutter Style Guide]:
https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo
[Features we expect every widget to implement]:
https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo#features-we-expect-every-widget-to-implement
[CLA]: https://cla.developers.google.com/
[flutter/tests]: https://github.com/flutter/tests
[breaking change policy]:
https://github.com/flutter/flutter/wiki/Tree-hygiene#handling-breaking-changes
[Discord]: https://github.com/flutter/flutter/wiki/Chat
@XilaiZhang
Copy link
Contributor

landed on stable

@nt4f04uNd
Copy link
Member Author

@XilaiZhang is there something left to do here?

@XilaiZhang
Copy link
Contributor

we are currently working on packaging and publishing the stable. once it is published then we can close this issue.

@github-actions
Copy link

github-actions bot commented Jun 1, 2023

This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of flutter doctor -v and a minimal reproduction of the issue.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cp: approved Approved cherry-pick request cp: merge-to-stable Cherry-picks that should be merged to stable cp: review Cherry-picks in the review queue
Projects
None yet
Development

No branches or pull requests

5 participants