-
Notifications
You must be signed in to change notification settings - Fork 286
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
KMW / 7577 Visits Per Visitor KMW Tile #7628
Conversation
…for widget sidebar.
…h other totals used in KMW tiles.
…wsKeyMetrics feature flag.
Note to reviewer, there is a known issue with failing PHP 7.4 WP Nightly build PHP tests. See Slack thread here. |
Build files for 81ab140 have been deleted. |
Size Change: +866 B (0%) Total Size: 1.41 MB
ℹ️ View Unchanged
|
Update on tests, 3 failures with known issues at present:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me 👍🏻
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Odd, I'm not seeing this widget appear in the list of Key Metrics tiles to select:
Any reason why this would be? It seems to work fine 🤔
EDIT: Forgot to enable newsKeyMetrics
😅 All good now! 🙂
Summary
Addresses issue:
Relevant technical choices
VisitsPerVisitorWidget
component and subsequent constants and identifiers such asKM_ANALYTICS_VISITS_PER_VISITOR
.newsKeyMetrics
feature flag.VisitsPerVisitorWidget
to list of key metrics widgets for side panel display.sessionsPerUser
metric is added to allowable shared metrics for shared dashboard scenarios.sessionsPerUser
toANALYTICS_4_METRIC_TYPES
const in order for correct mock data to be surfaced within story scenarios.PR Author Checklist
Do not alter or remove anything below. The following sections will be managed by moderators only.
Code Reviewer Checklist
Merge Reviewer Checklist