[Product • Global Unique Identifier] Tracking

toupper commented 1 week ago

Closes: #14375

With this PR we add two tracking events for the Product Global Unique Identifier project:

When editing the GTIN in the Product Inventory Settings
When scanning a barcode that matches the GTIN

We implement the necessary infrastructure to achieve these goals.

Description

Steps to reproduce

Global unique identifier edited in Product Inventory Settings

Go to Products
Tap on a product
Tap on Inventory
Edit the Global Unique Identifier and tap on Done
Check that the string is tracked:

🔵 Tracked product_inventory_settings_global_unique_identifier_field_edited, properties: [store_id: 3db14754-cf30-4c42-857d-d754bca1dac1, was_ecommerce_trial: false, is_wpcom_store: true, plan: business-bundle, site_url: https://americanwootester.wpcomstaging.com, blog_id: 214354650]

Product found by global unique identifier with the barcode scanner

Make sure you have a GTIN assigned to a product. Use https://barcode.tec-it.com/ to generate the barcode with that GTIN

Tap on Orders
Tap on the Barcode Scanner button (top left)
Scan the barcode with the assigned GTIN
See how this event is tracked:

🔵 Tracked product_search_via_global_unique_identifier_success, properties: [was_ecommerce_trial: false, store_id: 3db14754-cf30-4c42-857d-d754bca1dac1, plan: business-bundle, site_url: https://americanwootester.wpcomstaging.com, blog_id: 214354650, is_wpcom_store: true, source: order_list]

Testing information

I also tested:

When the GTIN is not edited (e.g. editing just the SKU and tapping on Done) the event above isn't tracked
When scanning a barcode that matches the SKU, the event above is not tracked

Screenshots

N/A

[X] I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

Reviewer (or Author, in the case of optional code reviews):

Please make sure these conditions are met before approving the PR, or request changes if the PR needs improvement:

[X] The PR is small and has a clear, single focus, or a valid explanation is provided in the description. If needed, please request to split it into smaller PRs.
[X] Ensure Adequate Unit Test Coverage: The changes are reasonably covered by unit tests or an explanation is provided in the PR description.
[X] Manual Testing: The author listed all the tests they ran, including smoke tests when needed (e.g., for refactorings). The reviewer confirmed that the PR works as expected on all devices (phone/tablet) and no regressions are added.

dangermattic commented 1 week ago

	1 Message
:book:	This PR contains changes to Tracks-related logic. Please ensure (author and reviewer) the following are completed: The tracks events must be validated in the Tracks system. Verify the internal Tracks spreadsheet has also been updated. Please consider registering any new events. The PR must be assigned the category: tracks label.

1 Message

:book:

This PR contains changes to Tracks-related logic. Please ensure (author and reviewer) the following are completed:

The tracks events must be validated in the Tracks system.
Verify the internal Tracks spreadsheet has also been updated.
Please consider registering any new events.
The PR must be assigned the category: tracks label.

Generated by :no_entry_sign: Danger

wpmobilebot commented 1 week ago

📲 You can test the changes from this Pull Request in WooCommerce iOS by scanning the QR code below to install the corresponding build.

	App Name	WooCommerce iOS
	Build Number	pr14377-ba7719f
	Version	21.1
	Bundle ID	com.automattic.alpha.woocommerce
	Commit	ba7719f28890b619e03fb9f000ed43ac609276e6
	App Center Build	WooCommerce - Prototype Builds #11537

Automatticians: You can use our internal self-serve MC tool to give yourself access to App Center if needed.

woocommerce / woocommerce-ios