Merge pull request #2069 from davidtwco/update-zulip-unsubscribe-api

marcoieni · web-flow · commit 2a65ad810a3c · 2025-11-11T14:18:14.000Z
zulip api: unsubscribe doesn't take an object
diff --git a/sync-team/src/zulip/api.rs b/sync-team/src/zulip/api.rs
@@ -241,9 +241,7 @@ impl ZulipApi {
         }
 
         if !remove_ids.is_empty() {
-            let subscriptions = serde_json::to_string(&serde_json::json!([{
-                "name": stream_name,
-            }]))?;
+            let subscriptions = serde_json::to_string(&serde_json::json!([stream_name]))?;
             let remove_ids = serialize_as_array(remove_ids);
             submit(reqwest::Method::DELETE, subscriptions, remove_ids)?;
         }

Original file line number	Diff line number	Diff line change
`@@ -241,9 +241,7 @@ impl ZulipApi {`
`241`	`241`	`}`
`242`	`242`
`243`	`243`	`if !remove_ids.is_empty() {`
`244`		`- let subscriptions = serde_json::to_string(&serde_json::json!([{`
`245`		`- "name": stream_name,`
`246`		`- }]))?;`
	`244`	`+ let subscriptions = serde_json::to_string(&serde_json::json!([stream_name]))?;`
`247`	`245`	`let remove_ids = serialize_as_array(remove_ids);`
`248`	`246`	`submit(reqwest::Method::DELETE, subscriptions, remove_ids)?;`
`249`	`247`	`}`