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
Unknown Object (File)
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
Branch
ashoat/registration
Lint
No Lint Coverage
Unit
No Test Coverage