dev: expand tracking#1083
Merged
Merged
Conversation
7b0fb99 to
5add9f1
Compare
d543cff to
a505477
Compare
There was a problem hiding this comment.
Pull Request Overview
This PR expands telemetry tracking by making it opt-in in the onboarding wizard, adding new usage events, and ensuring collected data is sent on uninstall.
- Add block-usage tracking in the Gutenberg editor and wire it into the telemetry system.
- Update the setup wizard UI to include an opt-in tracking notice and revised button text.
- Implement backend hooks to record import creation/run timestamps and counts, with data sent on deactivation.
Reviewed Changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| js/FeedzyLoop/tracking.js | Add block usage monitoring and analytics event emission. |
| js/FeedzyLoop/index.js | Import the new tracking module. |
| includes/views/css/style-wizard.css | Refactor wizard stylesheet and add styles for the new opt-in UI. |
| includes/layouts/setup-wizard.php | Inject opt-in tracking markup and update wizard button labels. |
| includes/feedzy-rss-feeds-deactivator.php | Trigger telemetry data send on plugin deactivation. |
| includes/admin/feedzy-rss-feeds-usage.php | Introduce a Feedzy_Rss_Feeds_Usage class to store and compute stats. |
| includes/admin/feedzy-rss-feeds-import.php | Hook usage tracking into import creation and run flows. |
| includes/admin/feedzy-rss-feeds-admin.php | Track import creation via post meta and set telemetry flag on opt-in. |
| includes/abstract/feedzy-rss-feeds-admin-abstract.php | Merge usage stats into the admin usage data output. |
Comments suppressed due to low confidence (2)
includes/admin/feedzy-rss-feeds-usage.php:1
- New usage tracking methods (
track_rss_import,track_import_creation, etc.) have been added. Consider adding unit or integration tests to cover these methods and validate their behavior over time.
<?php
js/FeedzyLoop/tracking.js:103
- The
subscribefunction from@wordpress/datatypically accepts only the listener callback and does not take a store parameter. Verify the API signature or usestore.subscribedirectly, or callsubscribe(() => updateBlockCounts())without the second argument.
const unsubscribe = subscribe(updateBlockCounts, blockEditorStore);
a505477 to
759e55c
Compare
Contributor
|
Plugin build for 4e78ca1 is ready 🛎️!
Note You can preview the changes in the Playground |
759e55c to
bc1fe4c
Compare
bc1fe4c to
48125cc
Compare
abaicus
approved these changes
Jul 15, 2025
abaicus
left a comment
There was a problem hiding this comment.
Looks good @Soare-Robert-Daniel. Just a small nitpick.
| $general_settings = array(); | ||
| $config = array(); | ||
|
|
||
| if ( $settings ) { |
There was a problem hiding this comment.
I think we can early-return here so we reduce nesting.
ca1bfa3 to
f2a8571
Compare
f2a8571 to
4e78ca1
Compare
Contributor
|
🎉 This PR is included in version 5.0.7 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
'feedzy_rss_feeds_logger_data'Note
The first time track usage will be available only for newer users after the release. Condition: install time lower than 24 hours.
Will affect visual aspect of the product
Yes
Screenshots
Improved boarding message with opt-in tracking
Metabase events for Block Usage
Metabase visualization for categories of user time between first import settings creation and first run.
Note
The time is registered only if the import run timestamp is lower than creation, which means older users who have daily crons running in the background will be unlikely to be counted. Part of
For projects running to avoid data from older users where we don't have the first new import click, we can enable this for projects that were created after the deployment.Test instructions
Settings > Enable Telemetry)Check before Pull Request is ready:
Closes https://github.com/Codeinwp/feedzy-rss-feeds-pro/issues/847
Closes https://github.com/Codeinwp/feedzy-rss-feeds-pro/issues/846