Page MenuHomePhabricator

[native] Add dev-only button to log out secondary device
ClosedPublic

Authored by bartek on May 28 2024, 4:25 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Nov 18, 6:14 AM
Unknown Object (File)
Mon, Nov 11, 9:41 AM
Unknown Object (File)
Mon, Nov 11, 8:35 AM
Unknown Object (File)
Sun, Nov 10, 10:07 PM
Unknown Object (File)
Sat, Nov 2, 12:43 AM
Unknown Object (File)
Sat, Nov 2, 12:42 AM
Unknown Object (File)
Sat, Nov 2, 12:42 AM
Unknown Object (File)
Sat, Nov 2, 12:31 AM
Subscribers

Details

Summary

Added a dev-only button to perform secondary device logout, for test purposes.
I deliberately did not include any checks if given device is primary or secondary, to test the hook behavior from previous diff.
In the future, logout button behavior will depend on whether current device is primary.

Depends on D12221

Test Plan

Pressed the button and repeated test plan from D12221 to check hook behavior.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bartek held this revision as a draft.
bartek published this revision for review.May 28 2024, 5:19 AM
bartek added inline comments.
native/profile/profile-screen.react.js
378 ↗(On Diff #40676)

Actually we don't need this TODO here because this is a testing-purpose button.

In the final version, a single logout button will trigger a correct action depending on device being primary or secondary

This revision is now accepted and ready to land.Jun 5 2024, 7:01 AM
native/profile/profile-screen.react.js
378

I thought we wanted to remove this TODO