jetpack icon indicating copy to clipboard operation
jetpack copied to clipboard

[DataSync] Clean up `DataSyncHooks.ts`

Open pyronaur opened this issue 1 year ago • 1 comments

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.

pyronaur avatar Feb 27 '24 07:02 pyronaur

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

  • It has been inactive for the past 6 months.
  • It hasn’t been labeled `[Pri] BLOCKER`, `[Pri] High`, `[Type] Feature Request`, `[Type] Enhancement`, `[Type] Janitorial`, `Good For Community`, `[Type] Good First Bug`, etc.

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.

github-actions[bot] avatar Aug 26 '24 00:08 github-actions[bot]