zulip / zulip-mobile

Zulip mobile apps for Android and iOS.
https://zulip.com/apps/
Apache License 2.0
1.3k stars 656 forks source link

notif: For test-notification button, require FL 234+ instead of 217+ #5819

Closed chrisbobbe closed 9 months ago

chrisbobbe commented 10 months ago

Servers with FL 217+ but not 234+ will send a payload to the bouncer that's missing realm_name, and the bouncer will error because it assumes realm_name is present. See API changelog: https://zulip.com/api/changelog

So, require 234+. Like 217, 234 was released in Zulip Server 8.0, so the error dialog we give the user should still be accurate:

Feature not available

This feature is only available for organizations on Zulip Server 8.0+.

gnprice commented 9 months ago

Thanks! Looks good; merging.