Skip to content

Add dashboard links to monthly pageview usage breakdown#6117

Merged
sanne-san merged 7 commits intomasterfrom
sanne-billing-dashboard-links
Mar 4, 2026
Merged

Add dashboard links to monthly pageview usage breakdown#6117
sanne-san merged 7 commits intomasterfrom
sanne-billing-dashboard-links

Conversation

@sanne-san
Copy link
Contributor

Changes

  • On the subscription settings page, each billing period's "Total billable pageviews" row and each individual site in the breakdown now show an external link icon that deep-links to the dashboard filtered to that exact billing cycle date range.
  • For the total link, the consolidated view domain is preferred when available; otherwise the site's own domain is used when the team has exactly one site.
CleanShot.2026-02-26.at.15.31.09.mp4

Tests

  • Automated tests have been added

Changelog

  • This PR does not make a user-facing change (not relevant to self-hosters)

Documentation

  • Docs have been updated

Dark mode

  • The UI has been tested both in dark and light mode

@sanne-san sanne-san force-pushed the sanne-billing-dashboard-links branch from 527cac4 to ad3e795 Compare March 2, 2026 13:34
- On the subscription settings page, each billing period's "Total billable pageviews" row and each individual site in the breakdown now show an external link icon that deep-links to the dashboard filtered to that exact billing cycle date range.
- For the total link, the consolidated view domain is preferred when available; otherwise the site's own domain is used when the team has exactly one site.
@sanne-san sanne-san force-pushed the sanne-billing-dashboard-links branch from ad3e795 to 1a6fc30 Compare March 2, 2026 13:35
@sanne-san sanne-san requested a review from a team March 3, 2026 10:28
RobertJoonas and others added 2 commits March 3, 2026 18:33
- Merge `site_domain` and `consolidated_view_domain` into a single`total_pageview_usage_domain` assign, resolving the precedence logic in the controller rather than the component
- Rename `sites` to `per_site` in the usage cycle map to distinguish it from the site count field in quota usage
- Remove `team_identifier` from controller assigns, template, and billing component — the session already carries the correct team context when the user is on the subscription settings page
@sanne-san sanne-san requested a review from RobertJoonas March 4, 2026 08:54
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.

Nice work! Left a suggestion inline that was not related to the recent changes but just something that I didn't notice in my initial review.

@sanne-san sanne-san added this pull request to the merge queue Mar 4, 2026
Merged via the queue into master with commit 5a26631 Mar 4, 2026
22 checks passed
@sanne-san sanne-san deleted the sanne-billing-dashboard-links branch March 4, 2026 12:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants