Page MenuHomePhabricator

[lib][web][native] Make possible to log out of one keyserver / delete account on one keyserver
ClosedPublic

Authored by inka on Jan 11 2024, 7:37 AM.
Tags
None
Referenced Files
Unknown Object (File)
Oct 16 2024, 3:26 AM
Unknown Object (File)
Oct 16 2024, 3:26 AM
Unknown Object (File)
Oct 16 2024, 3:26 AM
Unknown Object (File)
Oct 16 2024, 3:26 AM
Unknown Object (File)
Oct 16 2024, 3:25 AM
Unknown Object (File)
Oct 16 2024, 3:25 AM
Unknown Object (File)
Oct 11 2024, 10:05 AM
Unknown Object (File)
Sep 26 2024, 1:45 AM
Subscribers

Details

Summary

issue: ENG-5766
We want to be able to log out / delete account on a subset of keyservers
I'm also moving fetching preRequestUserState into useDeleteKeyserverAccount, to make the code clearer.

Test Plan

Tested that it is still possible to delete account and logout on both platforms

Diff Detail

Repository
rCOMM Comm
Branch
inka/login_reducers
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

inka requested review of this revision.Jan 11 2024, 8:00 AM
michal added inline comments.
lib/actions/user-actions.js
74–77

Can this be readonly?

lib/types/account-types.js
37–40

I think you need to wrap it in $ReadOnly<> as flow loses readonly on fields when you spread another object.

This revision is now accepted and ready to land.Jan 12 2024, 5:50 AM