Page MenuHomePhabricator
Differential D12513

[lib] Discard keyserver usernames for processServerRequestsActionType
ClosedPublic
Actions

Authored by inka on Jun 20 2024, 6:58 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Dec 19, 4:44 AM
Unknown Object (File)
Thu, Dec 19, 4:44 AM
Unknown Object (File)
Thu, Dec 19, 4:44 AM
Unknown Object (File)
Thu, Dec 19, 4:44 AM
Unknown Object (File)
Mon, Dec 16, 8:20 PM
Unknown Object (File)
Mon, Dec 9, 9:41 PM
Unknown Object (File)
Mon, Dec 9, 5:13 PM
Unknown Object (File)
Mon, Dec 9, 1:48 PM
View All 45 Files
Subscribers
ashoat

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

PathSize
lib/
reducers/
16 lines

Diff 41690

View Options

lib/reducers/user-reducer.js

Loading...