Kostas Seresiotis
Kostas Seresiotis
As discussed [here](https://github.com/saucal/pinterest-for-woocommerce/pull/7#pullrequestreview-646482552) This `useBodyClasses` hook is problematic and too tightly coupled to the components. The hook should not know anything about how/where it's used (e.g. `style` param and all...
There's increased coupling between `pinterest_data` option, `Crypto` class, and main plugin class. I think it would be clearer and simpler if: - Main plugin class is responsible for all option...
As discussed [here](https://github.com/saucal/pinterest-for-woocommerce/pull/7#pullrequestreview-646482552) > Need to justify why components need to affect the DOM outside the component (body classes etc) - this is unusual. Ideally the component wouldn't need to...
As discussed [here](https://github.com/saucal/pinterest-for-woocommerce/pull/7#pullrequestreview-646482552) > Some components are still using HOCs (e.g. withDispatch) - should consistently use hook style throughout the plugin.
The pin button is not working in product blocks - for example, the Storefront homepage layout that heavily uses blocks. We can handle that as follow up - this is...
As originally requested [here](https://github.com/saucal/pinterest-for-woocommerce/pull/19#discussion_r648736457) the new `All products` block should correctly trigger a tracking event when a product is added to the cart. Also related: #13
As discussed [here](https://github.com/saucal/pinterest-for-woocommerce/pull/19#pullrequestreview-681184031), ideally we should not print the tracking tag directly [as we currently do](https://github.com/saucal/pinterest-for-woocommerce/pull/19/files#diff-76b4e05a2bd981fd7244a32e751ea4b6bd3d4e0f63885d00d4ee915ab2fb6443R402). An ideal solution would also avoid the need for ignoring coding standards on those...