Page MenuHomePhabricator
Differential D12329

[lib] Dispatch processNewUserIDsActionType
ClosedPublic
Actions

Authored by inka on Thu, Jun 6, 3:49 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Jun 26, 8:22 PM
Unknown Object (File)
Tue, Jun 25, 6:46 AM
Unknown Object (File)
Sun, Jun 23, 12:10 PM
Unknown Object (File)
Thu, Jun 20, 9:00 AM
Unknown Object (File)
Tue, Jun 18, 5:26 PM
Unknown Object (File)
Sun, Jun 16, 10:41 PM
Unknown Object (File)
Fri, Jun 14, 8:03 AM
Unknown Object (File)
Fri, Jun 14, 6:44 AM
View All 24 Files
Subscribers
ashoat

Details

Reviewers
tomek
kamil
Commits
rCOMMa5cc4bae8fe7: [lib] Dispatch processNewUserIDsActionType
Summary

issue: [[https://linear.app/comm/issue/ENG-8190/[inka][june]-userstore-syncing-stop-processing-data-from-secondary | ENG-8190]]
After extracting user ids from server message we want to dispatch processNewUserIDsActionType to create entries for these ids in the user store. Then, UserInfosHandler will fetch missing user infos.

Test Plan

Tested that the action is dispatched both for socket messages and http api responses

Diff Detail

Repository
rCOMM Comm
Branch
inka/userStore
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

inka created this revision.Thu, Jun 6, 3:49 AM
Herald added a subscriber: ashoat. · View Herald TranscriptThu, Jun 6, 3:49 AM
inka added a parent revision: D12328: [lib] Avoid unnecessary state change in user reducer.Thu, Jun 6, 3:49 AM
Harbormaster completed remote builds in B29482: Diff 41035.Thu, Jun 6, 4:05 AM
inka requested review of this revision.Thu, Jun 6, 4:05 AM
tomek accepted this revision.Thu, Jun 6, 4:18 AM
tomek added inline comments.
lib/socket/user-infos-handler.react.js
14–35

Up to you, but we can useDispatch() here instead of passing the value as a prop

This revision is now accepted and ready to land.Thu, Jun 6, 4:18 AM
inka added a child revision: D12331: [lib] Stop processing user infos from secondary keyservers on joinThreadActionTypes and newThreadActionTypes.Thu, Jun 6, 4:22 AM
inka planned changes to this revision.Thu, Jun 6, 5:03 AM

Actually I should avoid dispatching the action if userIDs is empty. This results in a lot of unnecessary actions, mostly from PONG message

inka updated this revision to Diff 41039.Thu, Jun 6, 5:08 AM

Stop dispatching if array empty

This revision is now accepted and ready to land.Thu, Jun 6, 5:08 AM
Harbormaster completed remote builds in B29486: Diff 41039.Thu, Jun 6, 5:24 AM
ashoat added inline comments.Thu, Jun 6, 10:34 AM
lib/socket/user-infos-handler.react.js
14–35

Personally I think this would be cleaner

inka updated this revision to Diff 41106.Fri, Jun 7, 7:23 AM

Move dispatch

Harbormaster completed remote builds in B29530: Diff 41106.Fri, Jun 7, 7:39 AM
Closed by commit rCOMMa5cc4bae8fe7: [lib] Dispatch processNewUserIDsActionType (authored by inka). · Explain WhyTue, Jun 25, 1:21 AM
This revision was automatically updated to reflect the committed changes.
inka added a commit: rCOMMa5cc4bae8fe7: [lib] Dispatch processNewUserIDsActionType.

Revision Contents
Changeset List

Diff 41035

View Options

lib/keyserver-conn/call-single-keyserver-endpoint.js

Loading...
View Options

lib/socket/socket.react.js

Loading...
View Options

lib/socket/user-infos-handler.react.js

Loading...
View Options

lib/utils/user-info-extraction-utils.js

Loading...