Skip to content

Integrate consolidated view UI#5798

Merged
aerosol merged 37 commits intomasterfrom
integrate-consolidated-view-ui
Oct 22, 2025
Merged

Integrate consolidated view UI#5798
aerosol merged 37 commits intomasterfrom
integrate-consolidated-view-ui

Conversation

@aerosol
Copy link
Copy Markdown
Member

@aerosol aerosol commented Oct 13, 2025

Changes

This PR shows consolidated view on /sites.

  • currently only to super-admins (instead of putting it behind a feature flag)
  • allows early settings access, but this will remain mostly broken until Consolidated view settings #5795 is merged

Consolidated Views can be now enabled only to set-up teams. We'll have to go through what's already enabled on prod for testing purposes and adjust accordingly.

Seeds were adjusted to include secondary site. After re-seeding and creating a team, consolidated view can be enabled via customer support team inferface.

The PR builds upon #5805 and includes #5713 which never got into main branch by mistake.

Visuals - courtesy of @sanne-san 🙇

record-2025-10-19-14-51-50-were.jeer.hoff.mp4

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

@aerosol aerosol force-pushed the integrate-consolidated-view-ui branch 2 times, most recently from 99bfa52 to 0a250e6 Compare October 16, 2025 06:36
sanne-san and others added 12 commits October 19, 2025 14:22
- Improve existing site card layout
- Add static UI for the consolidated view
- Add dismissable upgrade card UI
- Extract favicon fetching logic to function
- Configure configurable fallback icon per route
- Add `/favicon/sources_globe/` route with different icon than `/favicon/sources/` to use on `/sites` page
* Minor query interface extension

* !fixup

* !fixup
@aerosol aerosol force-pushed the integrate-consolidated-view-ui branch from 26c27d5 to 192b27a Compare October 19, 2025 12:23
@github-actions
Copy link
Copy Markdown

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

@aerosol aerosol marked this pull request as ready for review October 19, 2025 13:05
@aerosol aerosol requested a review from a team October 19, 2025 13:06
@aerosol
Copy link
Copy Markdown
Member Author

aerosol commented Oct 20, 2025

There's something not entirely OK on the preview env:

https://pr-5798.review.plausible.io/sites

image

@aerosol aerosol added this pull request to the merge queue Oct 22, 2025
Merged via the queue into master with commit bce0890 Oct 22, 2025
16 checks passed
@aerosol aerosol deleted the integrate-consolidated-view-ui branch October 22, 2025 11:13
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.

3 participants