From 82e396dca606e3fcebe1db80aa37465291ecd3fb Mon Sep 17 00:00:00 2001 From: William Grant Date: Thu, 24 Aug 2023 11:02:45 +1000 Subject: [PATCH] feat: remove useEffect on right panel and use hook for checking is v2 is released --- .../conversation/right-panel/RightPanel.tsx | 30 ++----------------- ts/util/releaseFeature.ts | 5 ++++ 2 files changed, 7 insertions(+), 28 deletions(-) diff --git a/ts/components/conversation/right-panel/RightPanel.tsx b/ts/components/conversation/right-panel/RightPanel.tsx index b5969e889..d2a23ac00 100644 --- a/ts/components/conversation/right-panel/RightPanel.tsx +++ b/ts/components/conversation/right-panel/RightPanel.tsx @@ -1,42 +1,16 @@ -import React, { useEffect, useState } from 'react'; +import React from 'react'; import { useSelector } from 'react-redux'; import { getRightOverlayMode } from '../../../state/selectors/section'; -import { ReleasedFeatures } from '../../../util/releaseFeature'; import { OverlayDisappearingMessages } from './overlay/disappearing-messages/OverlayDisappearingMessages'; import { OverlayRightPanelSettings } from './overlay/OverlayRightPanelSettings'; const ClosableOverlay = () => { const rightOverlayMode = useSelector(getRightOverlayMode); - // TODO we can probably use the ReleasedFeatures.isDisappearMessageV2FeatureReleased instead here so we can remove the state. - const [showNewDisappearingMessageModes, setShowNewDisappearingMessageModes] = useState(false); - - useEffect(() => { - let isCancelled = false; - const checkFeature = async () => { - try { - const result = await ReleasedFeatures.checkIsDisappearMessageV2FeatureReleased(); - if (!isCancelled) { - setShowNewDisappearingMessageModes(result); - } - } catch (error) { - if (!isCancelled) { - window.log.debug(`ClosableOverlay checkFeature error: ${error}`); - } - } - }; - - void checkFeature(); - - return () => { - isCancelled = true; - }; - }, []); switch (rightOverlayMode) { case 'disappearing-messages': - // TODO legacy messages support will be removed in a future release - return ; + return ; default: return ; } diff --git a/ts/util/releaseFeature.ts b/ts/util/releaseFeature.ts index e371befa0..206598549 100644 --- a/ts/util/releaseFeature.ts +++ b/ts/util/releaseFeature.ts @@ -106,8 +106,13 @@ function isUserConfigFeatureReleasedCached(): boolean { return !!isUserConfigLibsessionFeatureReleased; } +function isDisappearMessageV2FeatureReleasedCached() { + return !!isDisappearingMessageFeatureReleased; +} + export const ReleasedFeatures = { checkIsUserConfigFeatureReleased, checkIsDisappearMessageV2FeatureReleased, isUserConfigFeatureReleasedCached, + isDisappearMessageV2FeatureReleasedCached, };