Page MenuHomePhabricator

[native] Extract onClose-disabling behavior from SIWEPanel to FullscreenSIWEPanel
ClosedPublic

Authored by ashoat on May 22 2023, 1:18 PM.
Tags
None
Referenced Files
F3373279: D7928.diff
Tue, Nov 26, 8:44 AM
Unknown Object (File)
Sun, Nov 10, 8:05 PM
Unknown Object (File)
Sun, Nov 10, 8:05 PM
Unknown Object (File)
Sun, Nov 10, 8:05 PM
Unknown Object (File)
Sun, Nov 10, 8:04 PM
Unknown Object (File)
Tue, Nov 5, 2:11 AM
Unknown Object (File)
Oct 17 2024, 8:24 AM
Unknown Object (File)
Oct 10 2024, 12:04 AM
Subscribers

Details

Summary

In the original SIWEPanel, we implemented logic to make sure that after a successful SIWE, we didn't trigger goBackToPrompt after the closing of the bottom sheet.

However, this logic is currently messing with the new implementation, where it's preventing a second SIWE authorization since the first one was never cleared since the onClose wasn't called.

This diff moves the behavior that blocks goBackToPrompt to FullscreenSIWEPanel, so that both use cases work correctly.

Depends on D7927

Test Plan
  1. I tested the old workflow and made sure that goBackToPrompt wasn't triggered after a successful SIWE
  2. See videos below for the impact on the new workflow:
beforeafter

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable