Store: Move product stats to store

[justinshreve]

Per our chat earlier, this branch movies the MC stat bump and track events away from being done server side (mu-plugins/tracks/hooks/woocommerce.php) to being handled via Calypso. The events are renamed and I'll keep the PHP hooks running for a bit so we can compare numbers (since it's our assumption the product count is inflated).

To Test:

• Run npm run test-client client/extensions/woocommerce and make sure all tests pass.
• Open your console and paste localStorage.setItem('debug', '*analytics*');
• Create a product and make sure you see both the analytic event and the 'bump stat' event.
• Update a product and make sure you see both the analytic event and the 'bump stat' event.
• Reset your setup flags and go through the address setup step, looking for calypso_woo_store_setup_country in your console.
• Following the FG page (linked on our P2), remove the blog sticker to make sure the stat bump is not blocked and retest.

[matticbot]

Test live in Docker at: https://dserve.a8c.com/?branch=update/store-product-stats

[timmyc]

I'm not seeing any debug statements when running on a test site. I can see tracks elsewhere in Store being discarded - like I deleted a product category and confirmed I could see the messaging then... but when editing a product I'm not seeing anything.

[timmyc]

so now with an action being returned we could even have some test coverage around test site stats being discarded correct?

[justinshreve]

Yup with some mocking. Though the import/reliance on the main 'lib/analytics' for the other method makes testing a bit hard because window is assumed to exist. Though if we are going to stop relying on our special track function we can clean that up a bit.

[timmyc]

Likewise when creating a new product, I'm not seeing any messaging:

[allendav]

Oooh - I like this

[timmyc]

When I actually checked out this branch, it tested out as expected for test and non test sites. 😄

[allendav]

If you can migrate this one too, that would close #21498 (and if not, no worries, I'll do it)

./app/dashboard/store-location-setup-view.js: this.props.bumpStat( 'calypso_woo_store_setup_country', this.state.address.country );

[allendav]

@justinshreve - I didn't see any code change in 71937b7 for the store location stat

[justinshreve]

@allendav I changed the import in that commit but otherwise its the same.

[allendav]

Works great. Also noted setup country stats bumping is properly discarded for test sites.