HomePhabricator
Diffusion Comm 9cb13605580b

[keyserver] Remove old current user info

Description

[keyserver] Remove old current user info

Summary:
ENG-4850
Depends on D9110
We can remove old current user info and simplify the code. Reasoning copied from the task:

  1. As you can see in D1463 when OldCurrentUserInfo the version was 86, while our current minimal supported version is 128
  2. fetchCurrentUserInfo uses fetchLoggedInUserInfo, which if you look at the code, never actually creates OldCurrentUserInfo structure, it always created the newer type
  3. Other code always uses fetchCurrentUserInfo as the source of truth so we can also change their signatures

Test Plan: yarn flow, analyze all changes and make sure that all data is in the end fetched using fetchCurrentUserInfo or from the clients

Reviewers: kamil, inka, atul, tomek

Reviewed By: tomek

Subscribers: ashoat, tomek

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

Details

Provenance
Michal Gniadek <michal.gniadek@swmansion.com>Authored on Sep 5 2023, 2:31 AM
Reviewer
tomek
Differential Revision
D9111: [keyserver] Remove old current user info
Parents
rCOMMebb529b9ae7b: Revert "[native] Temporary changes for staff release"
Branches
Unknown
Tags
Unknown

Event Timeline

Michal Gniadek <michal.gniadek@swmansion.com> committed rCOMM9cb13605580b: [keyserver] Remove old current user info (authored by Michal Gniadek <michal.gniadek@swmansion.com>).Sep 20 2023, 2:26 AM