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

Update the metadata for the survey pilot testing #2 #117

Merged
merged 2 commits into from
Oct 26, 2023

Conversation

jayoung-lee
Copy link
Contributor

Updated: ID, date, URL, condition, etc.
TODO: update the description if needed

Updated: ID, date, URL, condition, etc.
TODO: update the description
Copy link
Contributor

@eliasyishak eliasyishak left a comment

Choose a reason for hiding this comment

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

Sorry for the delayed response on this!

But LGTM, just curious though.. i'm just realizing that for us to use the flutter channel as a conditional, I am assuming that the channel that is reported by Flutter is beta.

Is this always how Flutter will report that channel? ie. can it vary in casing such as BETA, Beta, etc?

EDIT: Nevermind, I can confirm from the flutter tool code that it will always be lowercase

https://github.com/flutter/flutter/blob/master/packages/flutter_tools/lib/src/version.dart#L45-L50

Copy link
Contributor

@eliasyishak eliasyishak left a comment

Choose a reason for hiding this comment

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

Oh wait, you may want to also change the snooze period to be more than 5 minutes right?

@jayoung-lee
Copy link
Contributor Author

Updated!

@parlough parlough requested a review from eliasyishak October 26, 2023 18:27
Copy link
Contributor

@eliasyishak eliasyishak left a comment

Choose a reason for hiding this comment

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

LGTM :)

@jayoung-lee
Copy link
Contributor Author

Thank you :D

@jayoung-lee jayoung-lee merged commit 06056a2 into master Oct 26, 2023
3 checks passed
@jayoung-lee jayoung-lee deleted the jayoung-lee-patch-1 branch June 5, 2024 22:25
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.

2 participants