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

ServerCompatBanner: Bump minSupportedVersion to 2.1.0, as planned. #4962

Merged
merged 1 commit into from
Aug 25, 2021

Conversation

chrisbobbe
Copy link
Contributor

No description provided.

@chrisbobbe chrisbobbe requested review from gnprice and WesleyAC August 16, 2021 15:34
We haven't gotten negative feedback about the banner from the
smaller pool of users connecting to servers earlier than 2.0.0. The
banner was introduced with that threshold (in PR zulip#4750) to beta
users in v27.164 (2021-06-17) and to all users in v27.165
(2021-06-24).

So, proceed with aligning our threshold with what we declare in the
server stable-release doc, and update the code comment to match.
@gnprice
Copy link
Member

gnprice commented Aug 25, 2021

Thanks for following up on this! Looks good -- merging.

Here's the helpful details from your commit message on why it's time for this, just so they're more visible here:

We haven't gotten negative feedback about the banner from the
smaller pool of users connecting to servers earlier than 2.0.0. The
banner was introduced with that threshold (in PR #4750) to beta
users in v27.164 (2021-06-17) and to all users in v27.165
(2021-06-24).

@gnprice gnprice force-pushed the pr-bump-min-supported-version branch from 7aadde6 to 37074ce Compare August 25, 2021 19:35
@gnprice gnprice merged commit 37074ce into zulip:master Aug 25, 2021
@chrisbobbe
Copy link
Contributor Author

Thanks for the review and for copying over those details!

@chrisbobbe chrisbobbe deleted the pr-bump-min-supported-version branch November 4, 2021 21:42
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