-
Notifications
You must be signed in to change notification settings - Fork 29.7k
Suggest that people move to "beta" when they upgrade on "stable" #126972
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
Conversation
christopherfujino
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd like to vet the language a little before we merge... I like the idea though. This somehow feels a bit too strong -- do we really want to encourage users to migrate from stable to beta?
|
I thought this text was relatively neutral (just informing them that it exists and that they might want to use it), but I don't feel strongly about exactly what we say. What did you have in mind? |
|
I can see this being too successful and converting 80% of stable users to the beta channel, because it might be perceived as a recommendation. (Or, tongue in cheek: "Real experienced developers use the beta channel"!) I wonder about instead using a message on the |
|
I mean, if it's "too successful" we can always just remove the message! Since we have no particular targets I'm not especially concerned. In #126936 I'm adding channel descriptions for I'm not sure exactly when you're suggesting showing the message you suggest. Can you elaborate? |
|
I'm suggesting a commit very like the one that you're proposing here, but targeting Our recent data showed that |
|
Seems reasonable, let me whip up a new PR. |
|
Submitted a version of this for upgrading on "master" as #127146 |
|
I'd suggest we don't merge this one for now, and see how the other PR works out first. If we keep this, I'd probably simplify the text to something like, "For information on other Flutter release channels, visit foo.bar" |
|
Roger. |
…7146) Similar to #126972 but for master upgrades. Co-authored-by: Tim Sneath <[email protected]>
…tter#127146) Similar to flutter#126972 but for master upgrades. Co-authored-by: Tim Sneath <[email protected]>
No description provided.