HomePhabricator
Diffusion Comm 9b18b40e786e

[native] Don't prompt SIWE users about keeping login info on logout

Description

[native] Don't prompt SIWE users about keeping login info on logout

Summary:
Context: https://linear.app/comm/issue/ENG-2799/on-native-log-out-dont-prompt-siwe-users-about-keeping-login-info-in

Basically there are no native credentials to be kept or removed, so the existing Alert is irrelevant.

Instead added an "are you sure you want to log out" Alert since immediately logging out (destructive action) without any confirmation was a bit jarring.

Test Plan:
Here's how it looks when a SIWE user taps "Sign out":

Simulator Screen Shot - iPhone 14 Pro - 2023-02-21 at 23.32.16.png (2×1 px, 331 KB)

Here's how it looks when a "traditional" user taps "Sign out":

Simulator Screen Shot - iPhone 14 Pro - 2023-02-21 at 23.33.28.png (2×1 px, 500 KB)

Reviewers: ashoat, tomek

Reviewed By: ashoat

Differential Revision: https://phab.comm.dev/D6831

Details

Provenance
atulAuthored on Feb 22 2023, 2:58 PM
Reviewer
ashoat
Differential Revision
D6831: [native] Don't prompt SIWE users about keeping login info on logout
Parents
rCOMM192fdb2d72a7: [web] factor out app container position info logic out of tooltip utils
Branches
Unknown
Tags
Unknown