Page MenuHomePhabricator
Differential D9318

[lib] Move local id update reducer into specs
ClosedPublic
Actions

Authored by tomek on Sep 28 2023, 8:10 AM.
Tags
None
Referenced Files
F2151204: D9318.id31528.diff
Sun, Jun 30, 12:15 PM
F2150107: D9318.id.diff
Sun, Jun 30, 9:26 AM
F2148636: D9318.id31781.diff
Sun, Jun 30, 5:46 AM
F2145082: D9318.id31615.diff
Sat, Jun 29, 9:23 PM
Unknown Object (File)
Tue, Jun 25, 5:48 PM
Unknown Object (File)
Mon, Jun 24, 5:16 AM
Unknown Object (File)
Fri, Jun 21, 9:50 AM
Unknown Object (File)
Fri, Jun 21, 9:50 AM
View All 66 Files
Subscribers
ashoat
michal

Details

Reviewers
kamil
bartek
inka
Commits
rCOMMc9a66b403e9d: [lib] Move local id update reducer into specs
Summary

Move getting raw messages to a spec.

Depends on D9315

https://linear.app/comm/issue/ENG-4241/handle-processupdatesactiontype-as-a-part-of-a-spec

Test Plan

Check if the new code produces the same result as the old one.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

tomek created this revision.Sep 28 2023, 8:10 AM
Herald added a subscriber: ashoat. · View Herald TranscriptSep 28 2023, 8:10 AM
tomek added a child revision: D9319: [lib] Move message update reducer into specs.Sep 28 2023, 8:24 AM
Harbormaster completed remote builds in B22897: Diff 31528.Sep 28 2023, 8:28 AM
tomek requested review of this revision.Sep 28 2023, 8:28 AM
kamil accepted this revision.Sep 29 2023, 4:01 AM
kamil added inline comments.
lib/reducers/local-id-reducer.js
115–123 ↗(On Diff #31528)

as previously, I prefer a more functional approach here than a loop but I am afraid Flow will complain

This revision is now accepted and ready to land.Sep 29 2023, 4:01 AM
tomek updated this revision to Diff 31615.Oct 3 2023, 7:30 AM

Use more functional approach

Harbormaster completed remote builds in B22952: Diff 31615.Oct 3 2023, 7:48 AM
Closed by commit rCOMMc9a66b403e9d: [lib] Move local id update reducer into specs (authored by tomek). · Explain WhyOct 9 2023, 3:57 AM
This revision was automatically updated to reflect the committed changes.
tomek added a commit: rCOMMc9a66b403e9d: [lib] Move local id update reducer into specs.

Revision Contents
Changeset List

PathSize
lib/
reducers/
16 lines
shared/
updates/
3 lines
2 lines

Diff 31781

View Options

lib/reducers/local-id-reducer.js

Loading...
View Options

lib/shared/updates/join-thread-spec.js

Loading...
View Options

lib/shared/updates/update-spec.js

Loading...