From 9ba6c18e3ab7e4830bf5d8015c7f1361fb496208 Mon Sep 17 00:00:00 2001
From: evykassirer <evy@zulip.com>
Date: Sun, 29 Sep 2024 22:45:34 -0700
Subject: [PATCH] reload: Only add stream_id if it's defined.

This isn't a bug fix, but it does clean up the code. `reload_setup`
only uses the `stream_id` if it can parse it as an integer and
otherwise falls back to `undefined`, so it makes more sense to not
include undefined stream ids.
---
 web/src/reload.ts | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/web/src/reload.ts b/web/src/reload.ts
index 6ce80d6364..87af6ff7c3 100644
--- a/web/src/reload.ts
+++ b/web/src/reload.ts
@@ -59,7 +59,9 @@ function preserve_state(send_after_reload: boolean, save_compose: boolean): void
         if (msg_type === "stream") {
             const stream_id = compose_state.stream_id();
             url += "+msg_type=stream";
-            url += "+stream_id=" + (stream_id ? encodeURIComponent(stream_id) : undefined);
+            if (stream_id) {
+                url += "+stream_id=" + encodeURIComponent(stream_id);
+            }
             url += "+topic=" + encodeURIComponent(compose_state.topic());
         } else if (msg_type === "private") {
             url += "+msg_type=private";