Automattic / Automattic/jetpack

[DataSync] Clean up `DataSyncHooks.ts`

Open
#35,973 1 comment 0 reactions 1 assignee Claimed by @pyronaur View on GitHub
[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

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.