Remove unnecessary throw when processing snapshot (#636)

river-build · Aug 6, 2024 · 2be489a · 2be489a
1 parent 369c352
commit 2be489a
Showing 1 changed file with 8 additions and 6 deletions.
diff --git a/packages/sdk/src/streamStateView_Space.ts b/packages/sdk/src/streamStateView_Space.ts
@@ -223,15 +223,17 @@ export class StreamStateView_Space extends StreamStateView_AbstractContent {
                 }
                 break
             case ChannelOp.CO_UPDATED: {
+                // first take settings from payload, then from local channel, then defaults
                 const channel = this.spaceChannelsMetadata.get(channelId)
-                if (!channel) {
-                    throwWithCode(`Channel not found: ${channelId}`, Err.STREAM_BAD_EVENT)
-                }
+                const isDefault = isDefaultChannelId(channelId)
+                const isAutojoin = payload.settings?.autojoin ?? channel?.isAutojoin ?? isDefault
+                const hideUserJoinLeaveEvents =
+                    payload.settings?.hideUserJoinLeaveEvents ?? channel?.isAutojoin ?? false
                 this.spaceChannelsMetadata.set(channelId, {
-                    isDefault: isDefaultChannelId(channelId),
+                    isDefault,
                     updatedAtEventNum,
-                    isAutojoin: channel.isAutojoin,
-                    hideUserJoinLeaveEvents: channel.hideUserJoinLeaveEvents,
+                    isAutojoin,
+                    hideUserJoinLeaveEvents,
                 })
                 stateEmitter?.emit(
                     'spaceChannelUpdated',