Mirror
/
zulip
mirror of https://github.com/zulip/zulip.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

stream_data: Always initialize pin_to_top. 

This simplifies the StreamSubscription type, and parallels how we
always initialize every other property that isn’t in
NeverSubscribedStream (audible_notifications, desktop_notifications,
email_notifications, push_notifications, wildcard_mentions_notify,
color, is_muted).  email_address was already optional.

Signed-off-by: Anders Kaseorg <anders@zulip.com>
This commit is contained in:
Anders Kaseorg 2024-10-08 14:08:36 -07:00 committed by Tim Abbott
parent 44fde64c42
commit 9c2f38193d
4 changed files with 4 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
web/src/inbox_ui.ts
View File

@ -74,7 +74,7 @@ type StreamContext = {
invite_only: boolean;
is_web_public: boolean;
stream_name: string;
pin_to_top: boolean | undefined;
pin_to_top: boolean;
is_muted: boolean;
stream_color: string;
stream_header_color: string;

1
web/src/stream_data.ts
View File

@ -736,6 +736,7 @@ export function create_sub_from_server_data(
render_subscribers: !realm.realm_is_zephyr_mirror_realm || attrs.invite_only,
newly_subscribed: false,
is_muted: false,
pin_to_top: false,
desktop_notifications: null,
audible_notifications: null,
push_notifications: null,

2
web/src/stream_list.ts
View File

@ -471,7 +471,7 @@ class StreamSidebarRow {
}
update_whether_active(): void {
if (stream_list_sort.has_recent_activity(this.sub) || this.sub.pin_to_top === true) {
if (stream_list_sort.has_recent_activity(this.sub) || this.sub.pin_to_top) {
this.$list_item.removeClass("inactive_stream");
} else {
this.$list_item.addClass("inactive_stream");

8
web/src/sub_store.ts
View File

@ -9,8 +9,6 @@ import type {
stream_subscription_schema,
} from "./stream_types";
type PartialBy<T, K extends keyof T> = Omit<T, K> & Partial<T>;
export type Stream = z.infer<typeof stream_schema>;
export type StreamSpecificNotificationSettings = z.infer<
typeof stream_specific_notification_settings_schema
@ -28,11 +26,7 @@ export type ExtraStreamAttrs = {
};
// This is the actual type of subscription objects we use in the app.
export type StreamSubscription = PartialBy<
Omit<ApiStreamSubscription, "subscribers">,
"pin_to_top" | "email_address"
> &
ExtraStreamAttrs;
export type StreamSubscription = Omit<ApiStreamSubscription, "subscribers"> & ExtraStreamAttrs;
const subs_by_stream_id = new Map<number, StreamSubscription>();