Skip to content

Commit 191bbc0

Browse files
committed
api: Update Subscription for api changes
Remove `emailAddress` field per update in FL 266. Also changed `isWebPublic` to be non-null. It has been a field on subscriptions since at least FL 8 (see zulip/zulip@f93c19ec6 in `zerver/openapi/zulip.yaml` and `gather_subscriptions_helper` in `zerver/lib/actions.py`).
1 parent ae02444 commit 191bbc0

File tree

2 files changed

+2
-6
lines changed

2 files changed

+2
-6
lines changed

lib/api/model/model.dart

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -326,14 +326,13 @@ class Subscription {
326326
final int? streamWeeklyTraffic;
327327

328328
final bool inviteOnly;
329-
final bool? isWebPublic; // TODO(server-??): doc doesn't say when added
329+
final bool isWebPublic;
330330
final bool historyPublicToSubscribers;
331331
final int? messageRetentionDays;
332332
// final List<int> subscribers; // we register with includeSubscribers false
333333

334334
final StreamPostPolicy streamPostPolicy;
335335
// final bool? isAnnouncementOnly; // deprecated for `streamPostPolicy`; ignore
336-
final String emailAddress;
337336

338337
final int? canRemoveSubscribersGroupId; // TODO(server-6)
339338

@@ -366,7 +365,6 @@ class Subscription {
366365
required this.pushNotifications,
367366
required this.audibleNotifications,
368367
required this.pinToTop,
369-
required this.emailAddress,
370368
required this.isMuted,
371369
required this.isWebPublic,
372370
required this.color,

lib/api/model/model.g.dart

Lines changed: 1 addition & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)