Skip to content

api: Don't allow connecting to servers <5.0; show nag banner on <7.0 #5920

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

Merged
merged 1 commit into from
Apr 17, 2025

Conversation

chrisbobbe
Copy link
Contributor

@chrisbobbe chrisbobbe commented Apr 4, 2025

(The branch is misnamed; I realized after writing it that we can show the nag banner on 6.x too. 🙂)

api: Don't allow connecting to servers <5.0; show nag banner on <7.0

See 8552afce9 for the previous one of these.

The Zulip Server 5 release went out over 3 years ago, so the 4.x
releases are all long out of our announced 18-month compatibility
window:
  https://blog.zulip.com/2022/03/29/zulip-5-0-released/
  https://zulip.readthedocs.io/en/latest/overview/release-lifecycle.html#client-apps

So we can cheerfully refuse to connect to 4.x releases; our "Server
not supported" nag banner has been asking server admins on those
versions to upgrade since 8552afce9, which the changelog says went
out in v27.216, dated 2023-10-10.

The last 6.x release, 6.2, went out on 2023-05-19. That's also
outside the 18-month window, so it's time to start showing the nag
banner on 5.x and 6.x.

The last 7.x release, 7.5, went out on 2023-11-16, which is a month
and a bit shy of 18 months, so it's not yet time to show the banner
for versions 7.x.

@chrisbobbe chrisbobbe requested a review from gnprice April 4, 2025 08:51
@chrisbobbe chrisbobbe force-pushed the pr-show-nag-banner-on-5.x branch from 022a966 to 3fa3e37 Compare April 4, 2025 08:56
@chrisbobbe chrisbobbe changed the title api: Don't allow connecting to servers <5.0; show nag banner on <6.0 api: Don't allow connecting to servers <5.0; show nag banner on ~~<6.0~~ <7.0 Apr 4, 2025
@chrisbobbe chrisbobbe changed the title api: Don't allow connecting to servers <5.0; show nag banner on ~~<6.0~~ <7.0 api: Don't allow connecting to servers <5.0; show nag banner on <7.0 Apr 4, 2025
@chrisbobbe chrisbobbe force-pushed the pr-show-nag-banner-on-5.x branch from 3fa3e37 to 878252c Compare April 4, 2025 08:59
See 8552afc for the previous one of these.

The Zulip Server 5 release went out over 3 years ago, so the 4.x
releases are all long out of our announced 18-month compatibility
window:
  https://blog.zulip.com/2022/03/29/zulip-5-0-released/
  https://zulip.readthedocs.io/en/latest/overview/release-lifecycle.html#client-apps

So we can cheerfully refuse to connect to 4.x releases; our "Server
not supported" nag banner has been asking server admins on those
versions to upgrade since 8552afc, which the changelog says went
out in v27.216, dated 2023-10-10.

The last 6.x release, 6.2, went out on 2023-05-19. That's also
outside the 18-month window, so it's time to start showing the nag
banner on 5.x and 6.x.

The last 7.x release, 7.5, went out on 2023-11-16, which is a month
and a bit shy of 18 months, so it's not yet time to show the banner
for versions 7.x.
@chrisbobbe chrisbobbe force-pushed the pr-show-nag-banner-on-5.x branch from 878252c to 48f1e47 Compare April 4, 2025 09:00
@gnprice
Copy link
Member

gnprice commented Apr 17, 2025

Thanks for taking care of this! Looks good; merging.

@gnprice gnprice merged commit 48f1e47 into zulip:main Apr 17, 2025
1 check passed
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