Skip to content

feat: show weekly downloads on the extension detail page - #2135

Open
netomi wants to merge 3 commits into
split/download-analyticsfrom
split/weekly-downloads-card
Open

feat: show weekly downloads on the extension detail page#2135
netomi wants to merge 3 commits into
split/download-analyticsfrom
split/weekly-downloads-card

Conversation

@netomi

@netomi netomi commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Stack: 6 of 6. Split out of #2027. Base: #2134 (split/download-analytics).

All three commits are @gnugomez's, cherry-picked with -x, and they applied cleanly.

The visible end of the feature: a weekly-downloads card on the extension detail page, fed by the series endpoint from #2134.

  • weekly-downloads.tsx and use-extension-download-series.ts
  • the series call and its types in extension-registry-service.ts / extension-registry-types.ts
  • placed by extension-detail-overview.tsx

It reads analyticsEnabled from /api/version, so on a registry with analytics off nothing is requested and nothing is rendered — which is every deployment by default.

Why it is last

It is the only part of #2027 with a dependency in both directions: it needs the page primitives and Pill from #2130, and the series endpoint from #2134. Landing it earlier would have meant either stubbing the endpoint or shipping a card with nothing behind it.

Verification

yarn test 281 passing across 55 files (274 from #2130 plus 7 here), yarn lint clean, tsc --noEmit clean.

Stack summary

PR Unit Base
#2130 webui library surface main
#2131 ingestion move (pure relocation) #2130
#2132 ingestion rework #2131
#2133 time-series database #2132
#2134 analytics core and API #2133
#2135 this #2134

Two of those carry things worth deciding rather than just reviewing: #2132 is a rewrite of live download counting that #2027's "disabled by default" framing hides, and #2134 is where a synchronous time-series write lands on the download request path.

Refs #2027, #2025

@netomi
netomi force-pushed the split/weekly-downloads-card branch from 2736ee7 to 347270c Compare September 2, 2026 16:09
@netomi
netomi force-pushed the split/weekly-downloads-card branch from 40e2c5f to 5a246b6 Compare September 3, 2026 14:46
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