fix: do not add timer update when for off for groupv1

ts/models/conversation.ts

@@ -870,7 +870,6 @@ export class ConversationModel extends Backbone.Model<ConversationAttributes> {
     );
 
     // we don't add an update message when this comes from a config message, as we already have the SyncedMessage itself with the right timestamp to display
-    const shouldAddExpireUpdateMessage = !fromConfigMessage;
 
     if (this.isPublic()) {
       throw new Error("updateExpireTimer() Disappearing messages aren't supported in communities");
@@ -883,6 +882,13 @@ export class ConversationModel extends Backbone.Model<ConversationAttributes> {
       expirationMode = 'off';
       expireTimer = 0;
     }
+    const shouldAddExpireUpdateMessage =
+      (this.isPrivate() && !fromConfigMessage) ||
+      (this.isClosedGroup() &&
+        !PubKey.isClosedGroupV3(this.id) &&
+        !fromConfigMessage &&
+        expirationMode !== this.get('expirationMode') &&
+        expireTimer !== this.get('expireTimer'));
 
     // When we add a disappearing messages notification to the conversation, we want it
     // to be above the message that initiated that change, hence the subtraction.