Skip to content

Display rollup on /sites#5723

Closed
aerosol wants to merge 4 commits intorollup-dashboardfrom
display-rollups-on-sites-list
Closed

Display rollup on /sites#5723
aerosol wants to merge 4 commits intorollup-dashboardfrom
display-rollups-on-sites-list

Conversation

@aerosol
Copy link
Copy Markdown
Member

@aerosol aerosol commented Sep 15, 2025

Optional addition to #5719

image

@aerosol
Copy link
Copy Markdown
Member Author

aerosol commented Sep 15, 2025

@ukutaht @RobertJoonas if you feel this is good to have for the PoC, the PR is against the other branch

<ul class="my-6 grid grid-cols-1 gap-6 sm:grid-cols-2 lg:grid-cols-3">
<.site
:if={Plausible.Auth.is_super_admin?(@current_user) and @sites.total_entries > 1}
site={Plausible.Site.rollup(@current_team)}
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

this will become a feature flag I believe

end

defp merge_hourly_stats(hourly_stats) do
case hourly_stats do
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

clunky but once out of PoC, we'll revisit and add integration tests for it

@aerosol aerosol closed this Sep 15, 2025
@aerosol aerosol deleted the display-rollups-on-sites-list branch November 24, 2025 12:09
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.

1 participant