FIXME LINK api/notif [nfc]: Cut comment on FcmMessageChannelRecipient.streamId, relying on server 5+, FL 115+.

See "Feature level 115" from Zulip API changelog:

  https://zulip.com/api/changelog

See also:

  https://github.com/zulip/zulip/issues/ 18067

Signed-off-by: Zixuan James Li <[email protected]>

Author: PIG208

lib/api/notifications.dart

@@ -168,13 +168,9 @@ sealed class FcmMessageRecipient {
 @JsonSerializable(fieldRename: FieldRename.snake, createToJson: false)
 @_IntConverter()
 class FcmMessageChannelRecipient extends FcmMessageRecipient {
-  // Sending the stream ID in notifications is new in Zulip Server 5.
-  // But handling the lack of it would add complication, and we don't strictly
-  // need to -- we intend (#268) to cut pre-server-5 support before beta release.
-  // TODO(server-5): cut comment
   final int streamId;
 
-  // Current servers (as of 2023) always send the stream name.  But
+  // Current servers (as of 2024) always send the stream name.  But
   // future servers might not, once clients get the name from local data.
   // So might as well be ready.
   @JsonKey(name: 'stream')