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
Unknown Object (File)
Wed, Dec 25, 6:45 AM
Unknown Object (File)
Wed, Dec 25, 6:45 AM
Unknown Object (File)
Wed, Dec 25, 6:45 AM
Unknown Object (File)
Wed, Dec 25, 6:45 AM
Unknown Object (File)
Wed, Dec 18, 6:26 PM
Unknown Object (File)
Sat, Nov 30, 8:20 AM
Unknown Object (File)
Nov 6 2024, 9:17 AM
Unknown Object (File)
Oct 17 2024, 3:20 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
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