Page MenuHomePhabricator
Differential D11280

[lib] Add a reducer that handles successful ops processing
ClosedPublic
Actions

Authored by tomek on Mar 8 2024, 1:00 AM.
Tags
None
Referenced Files
F3354519: D11280.id38266.diff
Sat, Nov 23, 12:55 PM
F3354470: D11280.diff
Sat, Nov 23, 12:53 PM
Unknown Object (File)
Wed, Nov 13, 5:28 AM
Unknown Object (File)
Tue, Nov 12, 9:53 PM
Unknown Object (File)
Tue, Nov 12, 9:53 PM
Unknown Object (File)
Tue, Nov 12, 9:53 PM
Unknown Object (File)
Tue, Nov 12, 9:53 PM
Unknown Object (File)
Tue, Nov 12, 9:53 PM
View All 83 Files
Subscribers
ashoat

Details

Reviewers
kamil
inka
will
Commits
rCOMMbac8331a77bc: [lib] Add a reducer that handles successful ops processing
Summary

Remove the front of the queue when we are noticed that the processing was successful

https://linear.app/comm/issue/ENG-7102/reduce-the-action-this-should-filter-the-ops-out-from-the-array

Depends on D11279

Test Plan

Tests

Diff Detail

Repository
rCOMM Comm
Branch
ops
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

tomek created this revision.Mar 8 2024, 1:00 AM
Herald added a subscriber: ashoat. · View Herald TranscriptMar 8 2024, 1:00 AM
Harbormaster completed remote builds in B27398: Diff 37942.Mar 8 2024, 1:16 AM
tomek requested review of this revision.Mar 8 2024, 1:16 AM
tomek added a child revision: D11281: [lib] Add an optional ID field to all the actions.Mar 8 2024, 2:28 AM
kamil accepted this revision.Mar 11 2024, 4:13 AM
kamil added inline comments.
lib/reducers/db-ops-reducer.js
7
This revision is now accepted and ready to land.Mar 11 2024, 4:13 AM
tomek updated this revision to Diff 38261.Mar 21 2024, 5:04 AM

Rename, rebase

Harbormaster completed remote builds in B27640: Diff 38261.Mar 21 2024, 5:19 AM
tomek updated this revision to Diff 38266.Mar 21 2024, 8:08 AM

Simplify reducer logic

Harbormaster completed remote builds in B27645: Diff 38266.Mar 21 2024, 8:23 AM
tomek updated this revision to Diff 38367.Mar 27 2024, 8:45 AM

Simplify

tomek edited the summary of this revision. (Show Details)Mar 27 2024, 8:47 AM
Harbormaster completed remote builds in B27704: Diff 38367.Mar 27 2024, 9:01 AM
Closed by commit rCOMMbac8331a77bc: [lib] Add a reducer that handles successful ops processing (authored by tomek). · Explain WhyMar 28 2024, 8:11 AM
This revision was automatically updated to reflect the committed changes.
tomek added a commit: rCOMMbac8331a77bc: [lib] Add a reducer that handles successful ops processing.

Revision Contents
Changeset List

PathSize
lib/
reducers/
19 lines
62 lines
2 lines
types/
2 lines

Diff 37942

View Options

lib/reducers/db-ops-reducer.js

Loading...
View Options

lib/reducers/db-ops-reducer.test.js

Loading...
View Options

lib/reducers/master-reducer.js

Loading...
View Options

lib/types/db-ops-types.js

Loading...