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
F2191701: D10598.diff
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
Unknown Object (File)
May 31 2024, 3:03 AM
Unknown Object (File)
May 22 2024, 9:18 AM
Unknown Object (File)
May 22 2024, 9:18 AM
Unknown Object (File)
May 22 2024, 9:17 AM
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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

lib/keyserver-conn/call-keyserver-endpoint-provider.react.js
161 ↗(On Diff #35506)

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