mirrors/element-web
1
0
Fork 0
mirror of https://github.com/vector-im/element-web.git synced 2025-12-27 04:01:10 +01:00
Code Issues Projects Releases Wiki Activity

Switch to TypedEventEmitter and remove unnecessary loading state

Browse Source
This commit is contained in:
David Baker 2025-02-07 10:40:37 +01:00
parent 7c2d9f4954
commit cfd55a6887
1 changed files with 5 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/components/views/settings/tabs/user/EncryptionUserSettingsTab.tsx
View File

@ -23,8 +23,7 @@ import { SettingsSubheader } from "../../SettingsSubheader";
import { AdvancedPanel } from "../../encryption/AdvancedPanel";
import { ResetIdentityPanel } from "../../encryption/ResetIdentityPanel";
import { RecoveryPanelOutOfSync } from "../../encryption/RecoveryPanelOutOfSync";
import Spinner from "../../../elements/Spinner";
import { useEventEmitter } from "../../../../../hooks/useEventEmitter";
import { useTypedEventEmitter } from "../../../../../hooks/useEventEmitter";
import { KeyStoragePanel } from "../../encryption/KeyStoragePanel";
import { DeleteKeyStoragePanel } from "../../encryption/DeleteKeyStoragePanel";
@ -63,8 +62,8 @@ interface EncryptionUserSettingsTabProps {
}
const useKeyBackupIsEnabled = (): boolean | undefined => {
// The enabled state, or undefined when still loading
const [isEnabled, setIsEnabled] = useState<boolean | undefined>(undefined);
const [loading, setLoading] = useState(true);
const matrixClient = useMatrixClientContext();
@ -77,11 +76,10 @@ const useKeyBackupIsEnabled = (): boolean | undefined => {
useEffect(() => {
(async () => {
await checkStatus();
setLoading(false);
})();
}, [checkStatus, setLoading]);
}, [checkStatus]);
useEventEmitter(matrixClient, ClientEvent.AccountData, (event: MatrixEvent): void => {
useTypedEventEmitter(matrixClient, ClientEvent.AccountData, (event: MatrixEvent): void => {
const type = event.getType();
// Recheck the status if this account data has been updated as this implies it has changed
if (type === "m.org.matrix.custom.backup_disabled") {
@ -89,7 +87,7 @@ const useKeyBackupIsEnabled = (): boolean | undefined => {
}
});
return loading ? undefined : isEnabled;
return isEnabled;
};
/**