fix: make sure to note sent another message

in all remote cases

ts/models/conversation.ts

@@ -830,7 +830,7 @@ export class ConversationModel extends Backbone.Model<ConversationAttributes> {
     let expirationType = providedExpirationType;
     let expireTimer = providedExpireTimer;
     const lastDisappearingMessageChangeTimestamp = providedChangeTimestamp;
-    let source = providedSource;
+    const source = providedSource || UserUtils.getOurPubKeyStrFromCache();
 
     if (expirationType === undefined || expireTimer === undefined) {
       expirationType = 'off';
@@ -866,7 +866,6 @@ export class ConversationModel extends Backbone.Model<ConversationAttributes> {
     }
 
     const isOutgoing = Boolean(!receivedAt);
-    source = source || UserUtils.getOurPubKeyStrFromCache();
 
     // 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.
@@ -940,9 +939,9 @@ export class ConversationModel extends Backbone.Model<ConversationAttributes> {
     }
 
     // if change was made remotely, don't send it to the contact/group
-    if (fromSync || fromConfigMessage) {
+    if (receivedAt || fromSync || fromConfigMessage) {
       window.log.debug(
-        `WIP: updateExpireTimer() Not sending an ExpireTimerUpdate message because change was made remotely receivedAt:${receivedAt} fromSync:${fromSync} fromConfigMessage:${fromConfigMessage} `
+        `WIP: updateExpireTimer() We dont send an ExpireTimerUpdate because this was a remote change receivedAt:${receivedAt} fromSync:${fromSync} fromConfigMessage:${fromConfigMessage} `
       );
       return;
     }

ts/receiver/queuedJob.ts

@@ -438,6 +438,7 @@ export async function handleMessageJob(
             messageModel
           )}\n\nexpirationTimerUpdate: ${JSON.stringify(expirationTimerUpdate)}`
         );
+        confirm?.();
         return;
       }
 
@@ -456,6 +457,7 @@ export async function handleMessageJob(
             messageModel
           )}\n\nexpirationTimerUpdate: ${JSON.stringify(expirationTimerUpdate)}`
         );
+        confirm?.();
         return;
       }
 

ts/util/expiringMessages.ts

@@ -442,11 +442,12 @@ export async function checkForExpireUpdateInContentMessage(
     convoToUpdate.get('lastDisappearingMessageChangeTimestamp') >=
       lastDisappearingMessageChangeTimestamp
   ) {
-    window.log.info(
-      `WIP: checkForExpireUpdateInContentMessage() This is an outdated disappearing message setting. So we will ignore it.\ncontent: ${JSON.stringify(
-        content
-      )}\n\nconvoToUpdate: ${JSON.stringify(convoToUpdate)}`
-    );
+    // window.log.info(
+    //   `WIP: checkForExpireUpdateInContentMessage() This is an outdated disappearing message setting. So we will ignore it.\ncontent: ${JSON.stringify(
+    //     content
+    //   )}
+    // );
+
     return {
       expirationType: changeToDisappearingMessageType(
         convoToUpdate,