[web][native] Remove branching on `minimallyEncoded` in `redux-setup`
ClosedPublic
Actions

Authored by atul on Jan 24 2024, 10:24 PM.

Details

Reviewers

ashoat
ginsu
tomek

Commits

rCOMM71282321a570: [web][native] Remove branching on `minimallyEncoded` in `redux-setup`

Summary

We no longer need this branching since Redux thread store will always contain minimally encoded threads.

Depends on D10810

Test Plan

flow

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

atul created this revision.Jan 24 2024, 10:24 PM

Harbormaster completed remote builds in B26148: Diff 36124.Jan 24 2024, 10:43 PM

atul requested review of this revision.Jan 24 2024, 10:43 PM

atul added a child revision: D10812: Rename `useDisconnectedBarVisibilityHandler` to `useKeyserverReachabilityHandler`.Jan 24 2024, 10:47 PM

tomek accepted this revision.Jan 25 2024, 2:57 AM

This revision is now accepted and ready to land.Jan 25 2024, 2:57 AM

rebase

Harbormaster completed remote builds in B26289: Diff 36329.Jan 29 2024, 3:37 PM

Closed by commit rCOMM71282321a570: [web][native] Remove branching on `minimallyEncoded` in `redux-setup` (authored by atul). · Explain WhyJan 29 2024, 7:16 PM

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMM71282321a570: [web][native] Remove branching on `minimallyEncoded` in `redux-setup`.

Revision Contents
Changeset List

Path

Size

native/

redux/

redux-setup.js

79 lines

web/

redux/

redux-setup.js

39 lines

Diff 36329

View Options

native/redux/redux-setup.js