Skip to content

Render top referrers and search terms within the context of sources tabs#6204

Merged
zoldar merged 11 commits intomasterfrom
sources-referrers-google-preserve-other-tabs
Apr 2, 2026
Merged

Render top referrers and search terms within the context of sources tabs#6204
zoldar merged 11 commits intomasterfrom
sources-referrers-google-preserve-other-tabs

Conversation

@zoldar
Copy link
Copy Markdown
Contributor

@zoldar zoldar commented Mar 26, 2026

Changes

This PR reorganizes sources breakdown tabs so that "Top referrers" and "Search terms" (Google Search keywords) are always rendered in place of "Sources" instead of replacing the breakdown tile completely with a single tab.

I took this opportunity to add e2e coverage for search terms breakdown using mocked API response (which got updated to reflect actual real payload by the way).

Tests

  • Automated tests have been added

Dark mode

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

@zoldar zoldar force-pushed the sources-referrers-google-preserve-other-tabs branch from a93484e to 29b31d4 Compare March 30, 2026 13:17
@zoldar zoldar added the preview label Mar 30, 2026
@github-actions
Copy link
Copy Markdown

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

@zoldar zoldar force-pushed the sources-referrers-google-preserve-other-tabs branch 2 times, most recently from 7c644e9 to 6c0059b Compare March 31, 2026 14:53
@zoldar zoldar removed the preview label Apr 1, 2026
@zoldar zoldar force-pushed the sources-referrers-google-preserve-other-tabs branch 2 times, most recently from da45ddd to d71b64f Compare April 1, 2026 11:01
@zoldar zoldar force-pushed the sources-referrers-google-preserve-other-tabs branch from 134a3dd to 28f418a Compare April 1, 2026 12:57
@zoldar zoldar requested a review from a team April 1, 2026 12:59
@zoldar zoldar marked this pull request as ready for review April 1, 2026 12:59
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 1, 2026

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

Copy link
Copy Markdown
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.

Looks good to me! Thanks for further extending the E2E tests! 🙌

Comment thread assets/js/dashboard/stats/sources/index.js Outdated
Comment thread assets/js/dashboard/stats/sources/index.js Outdated
@zoldar zoldar added this pull request to the merge queue Apr 2, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 2, 2026
@zoldar zoldar added this pull request to the merge queue Apr 2, 2026
Merged via the queue into master with commit db9ab08 Apr 2, 2026
24 checks passed
@zoldar zoldar deleted the sources-referrers-google-preserve-other-tabs branch April 2, 2026 07:56
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