From 53da9ef0720f7b53eff11e4a39a548b039f1bba8 Mon Sep 17 00:00:00 2001 From: "neiljp (Neil Pilgrim)" Date: Fri, 25 Aug 2023 12:27:38 -0700 Subject: [PATCH] api-changelog: Clarify change in behavior at feature level 152. The change in behavior suggesting the need for this clarification is summarized below, as discussed in the following topic: https://chat.zulip.org/#narrow/stream/378-api-design/topic/update_message.20default.20notifications The new text is as suggested by Greg Price. The change at feature level 152 made the moving of topics within and between streams more consistent, by applying the optional parameters introduced for the latter to both cases (`send_notification_to_[new|old]_thread`). The default for the `old` notification value was changed to `false` at this point, but the equivalent `new` value was not. This introduced a change in behavior for existing clients of the API when moving topics within streams, which now always produce a notification message by default. The improved consistency and default behavior for new API clients were considered primary factors at the time of this change, overriding concerns over backwards compatbility. --- api_docs/changelog.md | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/api_docs/changelog.md b/api_docs/changelog.md index c94956a454..c9ff0065fb 100644 --- a/api_docs/changelog.md +++ b/api_docs/changelog.md @@ -493,12 +493,12 @@ No changes; feature level used for Zulip 6.0 release. **Feature level 152** -* [`PATCH /messages/{message_id}`](/api/update-message): The - `send_notification_to_old_thread` and - `send_notification_to_new_thread` parameters are now respected when - moving a topic within a stream. The default value for - `send_notification_to_old_thread` was changed from `true` to - `false`. +* [`PATCH /messages/{message_id}`](/api/update-message): + The default value for `send_notification_to_old_thread` was changed from + `true` to `false`. + When moving a topic within a stream, the `send_notification_to_old_thread` + and `send_notification_to_new_thread` parameters are now respected, and by + default a notification is sent to the new thread. **Feature level 151**