Automattic / jetpack

Security, performance, marketing, and design tools — Jetpack is made by WordPress experts to make WP sites safer and faster, and help you grow your traffic.
https://jetpack.com/
Other
1.59k stars 798 forks source link

[DataSync] Clean up `DataSyncHooks.ts` #35973

Open pyronaur opened 8 months ago

pyronaur commented 8 months ago

At the moment, the types are too verbose in there and the file is difficult to grok. DataSync is always going to be somewhat complicated, but I think the developer experience can probably be improved there so that it's better at documenting itself.

This probably can go hand in hand with writing DataSync documentation - having clear types is should make it easier to understand how to use DataSync.

Also, maybe each DataSync hook should get a dedicated file.

github-actions[bot] commented 2 months ago

This issue has been marked as stale. This happened because:

No further action is needed. But it's worth checking if this ticket has clear reproduction steps and it is still reproducible. Feel free to close this issue if you think it's not valid anymore — if you do, please add a brief explanation.