fix: closed groups works correctly now

I think

ts/receiver/queuedJob.ts

@@ -398,8 +398,6 @@ export async function handleMessageJob(
     // TODO I think this is incorrect? Let's fix
     if (
       conversation &&
-      (messageModel.isIncoming() || conversation.isMe()) &&
-      expirationType &&
       expireTimer > 0 &&
       Boolean(messageModel.get('expirationStartTimestamp')) === false
     ) {

ts/session/group/closed-group.ts

@@ -27,7 +27,6 @@ import { UserUtils } from '../utils';
 import { fromHexToArray, toHex } from '../utils/String';
 import {
   DisappearAfterSendOnly,
-  changeToDisappearingMessageConversationType,
   changeToDisappearingMessageType,
   setExpirationStartTimestamp,
 } from '../../util/expiringMessages';
@@ -233,8 +232,8 @@ function buildGroupDiff(convo: ConversationModel, update: GroupInfo): GroupDiff
   return groupDiff;
 }
 
-export async function updateOrCreateClosedGroup(details: GroupInfo, fromLegacyConfig?: boolean) {
+export async function updateOrCreateClosedGroup(details: GroupInfo) {
-  const { id, expirationType, expireTimer } = details;
+  const { id } = details;
 
   const conversation = await getConversationController().getOrCreateAndWait(
     id,
@@ -260,25 +259,6 @@ export async function updateOrCreateClosedGroup(details: GroupInfo, fromLegacyCo
     await conversation.updateGroupAdmins(details.admins, false);
   }
 
-  // if (
-  //   details.expirationType !== conversation.get('expirationType') ||
-  //   details.expireTimer !== conversation.get('expireTimer')
-  // ) {
-  //   await conversation.updateExpireTimer({
-  //     providedExpirationType: changeToDisappearingMessageConversationType(
-  //       conversation,
-  //       expirationType,
-  //       expireTimer
-  //     ),
-  //     providedExpireTimer: expireTimer || 0,
-  //     providedChangeTimestamp: GetNetworkTime.getNowWithNetworkOffset(),
-  //     providedSource: UserUtils.getOurPubKeyStrFromCache(),
-  //     receivedAt: Date.now(),
-  //     fromSync: true,
-  //     fromConfigMessage: Boolean(fromLegacyConfig),
-  //   });
-  // }
-
   await conversation.commit();
 }
 
@@ -324,6 +304,7 @@ async function sendAddedMembers(
   }
 
   const encryptionKeyPair = ECKeyPair.fromHexKeyPair(hexEncryptionKeyPair);
+  const expirationMode = convo.get('expirationType') || 'off';
   const existingExpireTimer = convo.get('expireTimer') || 0;
   // Send the Added Members message to the group (only members already in the group will get it)
   const closedGroupControlMessage = new ClosedGroupAddedMembersMessage({
@@ -346,6 +327,7 @@ async function sendAddedMembers(
     members,
     keypair: encryptionKeyPair,
     identifier: messageId || uuidv4(),
+    expirationType: changeToDisappearingMessageType(convo, expirationMode),
     expireTimer: existingExpireTimer,
   });
 

ts/util/expiringMessages.ts

@@ -374,7 +374,6 @@ export async function checkForExpireUpdateInContentMessage(
   const dataMessage = content.dataMessage as SignalService.DataMessage;
   // We will only support legacy disappearing messages for a short period before disappearing messages v2 is unlocked
   const isDisappearingMessagesV2Released = await ReleasedFeatures.checkIsDisappearMessageV2FeatureReleased();
-  // debugger;
 
   const isLegacyContentMessage = checkIsLegacyContentMessage(content);
   const isLegacyDataMessage = Boolean(