[lib] Return `mergeNewMessages(...)` directly where possible in `reduceMessageStore(...)`
ClosedPublic
Actions

Authored by atul on Jun 22 2022, 10:50 PM.

Details

Reviewers

• jacek
tomek
• abosh

Commits

rCOMM8a9bef17c383: [lib] Return `mergeNewMessages(...)` directly where possible in…

Summary

Basically D4337, but for all the other mergeNewMessages(...) callsites.

This is just a noop refactor, trying to simplify message-reducer

Depends on D4337

Test Plan

Close reading, flow

Diff Detail

Repository

rCOMM Comm

Branch

landjune23 (branched from master)

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

atul created this revision.Jun 22 2022, 10:50 PM

Herald added subscribers: • adrian, ashoat. · View Herald TranscriptJun 22 2022, 10:50 PM

atul edited the summary of this revision. (Show Details)Jun 22 2022, 10:50 PM

atul added a parent revision: D4337: [lib] Return result of `mergeNewMessages(...) immediately for `fetchMessagesBeforeCursor.success`/`fetchMostReecentMessages.success` action types.

Harbormaster completed remote builds in B9901: Diff 13693.Jun 22 2022, 10:55 PM

atul requested review of this revision.Jun 22 2022, 10:55 PM

atul added a child revision: D4339: [web][landing] Fix miscellaneous CSS issues.Jun 23 2022, 3:34 AM

tomek accepted this revision.Jun 23 2022, 11:52 AM

This revision is now accepted and ready to land.Jun 23 2022, 11:52 AM

• abosh accepted this revision.Jun 23 2022, 3:09 PM

rebase after cherrypicking and before landing

This revision was landed with ongoing or failed builds.Jun 24 2022, 5:47 PM

Closed by commit rCOMM8a9bef17c383: [lib] Return `mergeNewMessages(...)` directly where possible in…. · Explain Why

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMM8a9bef17c383: [lib] Return `mergeNewMessages(...)` directly where possible in….

Harbormaster completed remote builds in B9961: Diff 13776.Jun 24 2022, 5:52 PM

Revision Contents
Changeset List

Path

Size

lib/

reducers/

message-reducer.js

60 lines

Diff 13776

View Options

lib/reducers/message-reducer.js

[lib] Return `mergeNewMessages(...)` directly where possible in `reduceMessageStore(...)`ClosedPublicActions