Page MenuHomePhabricator

[lib][native] Move all the Redux-binding code out of action-utils.js
ClosedPublic

Authored by ashoat on Jan 10 2024, 12:18 PM.
Tags
None
Referenced Files
F2214122: D10598.diff
Mon, Jul 8, 3:41 PM
F2209730: D10598.diff
Sun, Jul 7, 6:36 PM
F2209624: D10598.diff
Sun, Jul 7, 6:07 PM
Unknown Object (File)
Fri, Jul 5, 5:18 PM
Unknown Object (File)
Thu, Jul 4, 4:53 PM
Unknown Object (File)
Tue, Jul 2, 11:12 PM
Unknown Object (File)
Tue, Jul 2, 1:54 PM
Unknown Object (File)
Sat, Jun 29, 11:33 PM
Subscribers
None

Details

Summary

Moving this into a file named lib/keyserver-conn/call-keyserver-endpoint-provider.react.js.

While this diff is a pure move (with one exception, see inline comment), in a later diff I will refactor the code into a provider pattern. I decided to avoid needing a rename later, which is why the filename references a provider but there is no actual provider there.

Depends on D10597

Test Plan

Flow

Diff Detail

Repository
rCOMM Comm
Branch
ashoat/ident
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

lib/keyserver-conn/call-keyserver-endpoint-provider.react.js
161

This diff is a pure move with the exception of this line. I found no uses of this type in the codebase, so I decided to delete it

Harbormaster returned this revision to the author for changes because remote builds failed.Jan 10 2024, 12:26 PM
Harbormaster failed remote builds in B25682: Diff 35506!
This revision is now accepted and ready to land.Jan 11 2024, 2:31 AM