Page MenuHomePhabricator

D13772.id45356.diff
No OneTemporary

D13772.id45356.diff

diff --git a/native/account/siwe-panel.react.js b/native/account/siwe-panel.react.js
--- a/native/account/siwe-panel.react.js
+++ b/native/account/siwe-panel.react.js
@@ -91,6 +91,17 @@
// to be unmounted/remounted by our parent component prior to a retry
const nonceNotNeededRef = React.useRef(false);
+ // When we succeed or fail, we call onSuccessOrCancel, which cancels
+ // any pending setWalletConnectModalHeight callbacks
+ const walletConnectModalHeightResetTimeout = React.useRef<?TimeoutID>();
+ const onSuccessOrCancel = React.useCallback(() => {
+ nonceNotNeededRef.current = true;
+ if (walletConnectModalHeightResetTimeout.current) {
+ clearTimeout(walletConnectModalHeightResetTimeout.current);
+ walletConnectModalHeightResetTimeout.current = undefined;
+ }
+ }, []);
+
React.useEffect(() => {
if (siweNonce && siweStatement && siweIssuedAt) {
setNonceInfo({
@@ -101,7 +112,7 @@
const siwePrimaryIdentityPublicKey =
getPublicKeyFromSIWEStatement(siweStatement);
setPrimaryIdentityPublicKey(siwePrimaryIdentityPublicKey);
- nonceNotNeededRef.current = true;
+ onSuccessOrCancel();
return;
}
if (nonceNotNeededRef.current) {
@@ -123,7 +134,7 @@
{
text: 'OK',
onPress: () => {
- nonceNotNeededRef.current = true;
+ onSuccessOrCancel();
onClosing();
},
},
@@ -158,6 +169,7 @@
siweNonce,
siweStatement,
siweIssuedAt,
+ onSuccessOrCancel,
]);
const [isLoading, setLoading] = React.useState(true);
@@ -200,7 +212,7 @@
if (data.type === 'siwe_success') {
const { address, message, signature } = data;
if (address && signature) {
- nonceNotNeededRef.current = true;
+ onSuccessOrCancel();
closeBottomSheet?.();
await onSuccessfulWalletSignature({
address,
@@ -210,14 +222,23 @@
});
}
} else if (data.type === 'siwe_closed') {
- nonceNotNeededRef.current = true;
+ onSuccessOrCancel();
onClosing();
closeBottomSheet?.();
} else if (data.type === 'walletconnect_modal_update') {
+ if (nonceNotNeededRef.current) {
+ return;
+ }
const height = data.state === 'open' ? data.height : 0;
if (!walletConnectModalHeight || height > 0) {
setWalletConnectModalHeight(height);
+ return;
}
+ walletConnectModalHeightResetTimeout.current = setTimeout(() => {
+ if (!nonceNotNeededRef.current) {
+ setWalletConnectModalHeight(height);
+ }
+ }, 50);
}
},
[
@@ -226,16 +247,17 @@
closeBottomSheet,
walletConnectModalHeight,
nonceTimestamp,
+ onSuccessOrCancel,
],
);
const prevClosingRef = React.useRef<?boolean>();
React.useEffect(() => {
if (closing && !prevClosingRef.current) {
- nonceNotNeededRef.current = true;
+ onSuccessOrCancel();
closeBottomSheet?.();
}
prevClosingRef.current = closing;
- }, [closing, closeBottomSheet]);
+ }, [closing, closeBottomSheet, onSuccessOrCancel]);
const nonce = nonceInfo?.nonce;
const issuedAt = nonceInfo?.issuedAt;

File Metadata

Mime Type
text/plain
Expires
Sat, Oct 26, 8:12 PM (21 h, 39 m)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
2367590
Default Alt Text
D13772.id45356.diff (3 KB)

Event Timeline