Skip to content
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

Remove total_visitors metric from dashboard breakdown reports #5058

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

macobo
Copy link
Contributor

@macobo macobo commented Feb 10, 2025

Changes

This metric was only shown in some details modals and with scroll depth goals incoming IMO cause confusion and complexity.

Long-term we plan to show the fraction(s) in tooltips.

Follow-up to #4999

Tests

  • Automated tests have been added
  • This PR does not require tests

Changelog

  • Entry has been added to changelog
  • This PR does not make a user-facing change

Documentation

  • Docs have been updated
  • This change does not need a documentation update

Dark mode

  • The UI has been tested both in dark and light mode
  • This PR does not change the UI

This metric was only shown in some details modals and with scroll depth
goals incoming cause confusion and complexity.

Long-term we plan to show the fraction(s) in tooltips.
Copy link

Preview environment👷🏼‍♀️🏗️
PR-5058

Copy link
Contributor

@RobertJoonas RobertJoonas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Little nit to make the changelog entry more specific. Otherwise looks good to me! (Took the liberty to merge master into your branch and did a quick staging check too)

@@ -13,6 +13,7 @@ All notable changes to this project will be documented in this file.

- Internal stats API routes no longer support legacy dashboard filter format.
- Dashboard no longer shows "Unique visitors" in top stats when filtering by a goal which used to count all users including ones who didn't complete the goal. "Unique conversions" shows the number of unique visitors who completed the goal.
- Dashboard no longer shows "Total visitors" in breakdown details modals.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- Dashboard no longer shows "Total visitors" in breakdown details modals.
- Goal-filtered dashboard no longer shows "Total visitors" in breakdown details modals.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants