Automattic / Automattic/jetpack
[DataSync] Clean up `DataSyncHooks.ts`
Open
[JS Package] React Data Sync Client
[Status] Stale
- Dominant language
- PHP
- Stars
- 1.8k
- Forks
- 898
- Avg merge
- 1d 18h
- Merged PRs (30d)
- 774
Description
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.
Contributor guide
Assessment
This issue has not been assessed yet.