[lib] Discard keyserver usernames for processServerRequestsActionType
ClosedPublic
Actions

Authored by inka on Jun 20 2024, 6:58 AM.

Details

Reviewers

tomek
kamil

Commits

rCOMMc251b8874c5e: [lib] Discard keyserver usernames for processServerRequestsActionType

Summary

issue: ENG-8365

Test Plan

Tested that if processServerRequestsActionType is dispatched with some userInfos in stateChanges and those users are not in the store, the username is set to null

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

inka created this revision.Jun 20 2024, 6:58 AM

Herald added a subscriber: ashoat. · View Herald TranscriptJun 20 2024, 6:58 AM

inka added a parent revision: D12512: [lib] Discard keyserver usernames for incremental state sync and processUpdatesActionType.Jun 20 2024, 6:58 AM

inka added a child revision: D12514: [lib] Discard keyserver usernames for updateUserAvatarActionTypes.Jun 20 2024, 7:02 AM

Harbormaster completed remote builds in B29835: Diff 41570.Jun 20 2024, 7:15 AM

inka requested review of this revision.Jun 20 2024, 7:15 AM

kamil accepted this revision.Jun 21 2024, 3:28 AM

kamil added inline comments.

lib/reducers/user-reducer.js
545 ↗	(On Diff #41570)	is this spread needed?

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

inka added inline comments.Jun 21 2024, 5:14 AM

lib/reducers/user-reducer.js
545 ↗	(On Diff #41570)	Yes, because `convertUserInfosToReplaceUserOps` returns a read only array and we want to push to this array in the next lines

Closed by commit rCOMMc251b8874c5e: [lib] Discard keyserver usernames for processServerRequestsActionType (authored by inka). · Explain WhyJun 25 2024, 1:22 AM

This revision was automatically updated to reflect the committed changes.

inka added a commit: rCOMMc251b8874c5e: [lib] Discard keyserver usernames for processServerRequestsActionType.

Revision Contents
Changeset List

Path

Size

lib/

reducers/

user-reducer.js

16 lines

Diff 41690

View Options

lib/reducers/user-reducer.js

[lib] Discard keyserver usernames for processServerRequestsActionTypeClosedPublicActions