Page MenuHomePhabricator

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

Authored by ashoat on Mon, May 22, 1:18 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Jun 2, 3:41 PM
Unknown Object (File)
Fri, Jun 2, 7:03 AM
Unknown Object (File)
Mon, May 29, 8:42 PM
Unknown Object (File)
Mon, May 22, 7:16 PM
F552074: Screen Recording 2023-05-22 at 4.11.53 PM.mov
Mon, May 22, 1:19 PM
F552073: Screen Recording 2023-05-22 at 4.11.10 PM.mov
Mon, May 22, 1:19 PM
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