Page MenuHomePhabricator

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

Authored by bartek on May 28 2024, 4:26 AM.
Tags
None
Referenced Files
F2198095: D12223.id41028.diff
Fri, Jul 5, 4:41 PM
F2198006: D12223.id40677.diff
Fri, Jul 5, 4:27 PM
Unknown Object (File)
Thu, Jul 4, 8:14 AM
Unknown Object (File)
Wed, Jul 3, 6:47 AM
Unknown Object (File)
Fri, Jun 28, 2:06 AM
Unknown Object (File)
Thu, Jun 27, 11:19 AM
Unknown Object (File)
Thu, Jun 27, 11:18 AM
Unknown Object (File)
Thu, Jun 27, 11:18 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 D12222

Test Plan

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

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

bartek held this revision as a draft.
bartek published this revision for review.May 28 2024, 5:19 AM
kamil added inline comments.
web/settings/account-settings.react.js
158

this flag is not dev-only - but also for staff members, is it intentional?

This revision is now accepted and ready to land.Thu, Jun 6, 12:35 AM
web/settings/account-settings.react.js
158

Yeah should be dev only, I'll fix that