Page MenuHomePhabricator
Differential D9826

[lib/native] extend keyserverAdminUsername to be keyserverAdminUserInfo
ClosedPublic
Actions

Authored by ginsu on Nov 10 2023, 11:15 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 20, 7:28 PM
Unknown Object (File)
Tue, Dec 10, 3:57 PM
Unknown Object (File)
Mon, Nov 25, 5:39 AM
Unknown Object (File)
Nov 15 2024, 4:51 AM
Unknown Object (File)
Nov 15 2024, 2:45 AM
Unknown Object (File)
Nov 15 2024, 2:38 AM
Unknown Object (File)
Nov 15 2024, 12:41 AM
Unknown Object (File)
Nov 8 2024, 4:11 AM
View All 66 Files
Subscribers
ashoat
tomek
will

Details

Reviewers
inka
rohan
michal
Commits
rCOMM371f6f808e5e: [lib/native] extend keyserverAdminUsername to be keyserverAdminUserInfo
Summary

The remove keyserver action needs the keyserver admin user id. I realized as I was implementing the solution to remove a keyserver from the keyserver store that I am only had the username available to me. This diff extends the keyserverAdminUsername to be keyserverAdminUserInfo which also holds the keyserver admin user id

Depends on D9806

Test Plan

flow and confirmed that I now have the userID in the keyserverAdminUserInfo

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

ginsu created this revision.Nov 10 2023, 11:15 AM
Herald added subscribers: will, tomek, ashoat. · View Herald TranscriptNov 10 2023, 11:15 AM
ginsu updated this revision to Diff 33077.Nov 10 2023, 11:18 AM

update

ginsu edited the summary of this revision. (Show Details)Nov 10 2023, 11:19 AM
ginsu added reviewers: inka, rohan, michal.
ginsu added a parent revision: D9806: [lib] introduce removeKeyserverActionType.
ginsu added a child revision: D9828: [native] introduce temp onPressDisconnectKeyserver callback.Nov 10 2023, 11:31 AM
Harbormaster completed remote builds in B24003: Diff 33077.Nov 10 2023, 11:47 AM
ginsu requested review of this revision.Nov 10 2023, 11:47 AM
Harbormaster completed remote builds in B24002: Diff 33076.Nov 10 2023, 11:50 AM
inka accepted this revision.Nov 14 2023, 12:26 AM
This revision is now accepted and ready to land.Nov 14 2023, 12:26 AM
ginsu updated this revision to Diff 33238.Nov 14 2023, 5:25 PM

rebase before landing

This revision was landed with ongoing or failed builds.Nov 14 2023, 5:47 PM
Closed by commit rCOMM371f6f808e5e: [lib/native] extend keyserverAdminUsername to be keyserverAdminUserInfo (authored by ginsu). · Explain Why
This revision was automatically updated to reflect the committed changes.
ginsu added a commit: rCOMM371f6f808e5e: [lib/native] extend keyserverAdminUsername to be keyserverAdminUserInfo.
Harbormaster failed remote builds in B24131: Diff 33238!Nov 14 2023, 5:57 PM
Harbormaster completed remote builds in B24131: Diff 33238.Nov 14 2023, 6:51 PM

Revision Contents
Changeset List

Diff 33077

View Options

lib/selectors/keyserver-selectors.js

Loading...
View Options

lib/types/keyserver-types.js

Loading...
View Options

native/profile/keyserver-selection-bottom-sheet.react.js

Loading...
View Options

native/profile/keyserver-selection-list-item.react.js

Loading...
View Options

native/profile/keyserver-selection-list.react.js

Loading...