[lib] Discard keyserver usernames for full state sync
ClosedPublic
Actions

Authored by inka on Thu, Jun 20, 4:34 AM.

Details

Reviewers

tomek
kamil

Commits

rCOMM96396357b076: [lib] Discard keyserver usernames for full state sync

Summary

issue: ENG-8365

Test Plan

Checked that if users are not present in the store, their user infos are added without usernames. Checked that if they are present, their usernames don't change

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

inka created this revision.Thu, Jun 20, 4:34 AM

Herald added a subscriber: ashoat. · View Herald TranscriptThu, Jun 20, 4:34 AM

inka added a parent revision: D12509: [lib] Discard keyserver usernames for keyserverAuthActionTypes.Thu, Jun 20, 4:35 AM

Harbormaster completed remote builds in B29824: Diff 41557.Thu, Jun 20, 4:57 AM

inka requested review of this revision.Thu, Jun 20, 4:57 AM

inka added a child revision: D12512: [lib] Discard keyserver usernames for incremental state sync and processUpdatesActionType.Thu, Jun 20, 4:57 AM

kamil accepted this revision.Fri, Jun 21, 3:25 AM

This revision is now accepted and ready to land.Fri, Jun 21, 3:25 AM

Closed by commit rCOMM96396357b076: [lib] Discard keyserver usernames for full state sync (authored by inka). · Explain WhyTue, Jun 25, 1:22 AM

This revision was automatically updated to reflect the committed changes.

inka added a commit: rCOMM96396357b076: [lib] Discard keyserver usernames for full state sync.

What happens if the client first becomes aware of a particular user in fullStateSyncActionType? Will we commit the ID and avatar to UserStore (but not the username), and then later fetch the username from identity?

I spoke to @inka about this in a 1:1 and they confirmed my assessment above.

Revision Contents
Changeset List

Path

Size

lib/

reducers/

user-reducer.js

6 lines

Diff 41557

View Options

lib/reducers/user-reducer.js

[lib] Discard keyserver usernames for full state syncClosedPublicActions