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

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

Merged
merged 1 commit into from
Feb 16, 2024

Commits on Feb 16, 2024

  1. notif: For test-notification button, require FL 234+ instead of 217+

    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+.
    chrisbobbe authored and gnprice committed Feb 16, 2024
    Configuration menu
    Copy the full SHA
    838c58e View commit details
    Browse the repository at this point in the history