search

Automattic / wp-calypso

The JavaScript and API powered WordPress.com
https://developer.wordpress.com
GNU General Public License v2.0
12.4k stars 1.98k forks source link

Refactor: Simplify client/blocks/importer/hooks/use-site-can-migrate.ts custom hook #84344

Open bogiii opened 10 months ago

bogiii commented 10 months ago

This file is kind of hard to follow... Let's refactor it and provide better naming.

https://github.com/Automattic/wp-calypso/blob/31ab100b87f326795096d2029dae10605d71c77b/client/blocks/importer/hooks/use-site-can-migrate.ts

flootr commented 10 months ago

Regarding my feedback here https://github.com/Automattic/wp-calypso/pull/83593#discussion_r1392310244 I think we should not be syncing state from any query hooks if possible.