feat(ai): render ChatGPT Team (Codex + Chat) bullets in ai_adoption#160
feat(ai): render ChatGPT Team (Codex + Chat) bullets in ai_adoption#160mozhaev-dev wants to merge 1 commit into
Conversation
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
Reviewed together with the backend PR (constructorfabric/insight#1282). The placement is consistent with the
|
FE-only layout placement for the ChatGPT Team metric_keys served by the analytics-api metric_catalog (insight #1282): chatgpt_active → ai_members, codex_lines / codex_sessions → ai_codex_detail, with matching display-order entries. Labels/sublabels/thresholds come from the wire catalog (post-#82, threshold-config.ts deleted). Rebased onto current main as a single signed commit (DCO). Part of constructorfabric/insight#497. Signed-off-by: Sergey Mozhaev <Sergey.Mozhaev@constructor.tech>
d1b8061 to
98c2174
Compare
Part of constructorfabric/insight#497
Render ChatGPT Team (Codex + Chat) bullets in ai_adoption
Adds the FE-only layout placement for the three new ChatGPT Team bullet metrics emitted by the analytics-api
metric_catalogseed (insight PRfeat/chatgpt-team-connector→m20260609_000002).Since #66 wave 4 (
199f2da) deleted the compile-inthreshold-config.tsand made the wiremetric_catalogthe sole source of metric metadata (labels / sublabels / thresholds), this PR carries only layout composition — no metric metadata. Six lines insrc/lib/insight/bullet-layout-groups.ts:BULLET_LAYOUT_GROUPS:chatgpt_active → ai_members,codex_lines/codex_sessions → ai_codex_detail.BULLET_DISPLAY_ORDER: the same three keys placed alongside the existing Codex / Claude-Code entries in theai_adoptionblock.The labels, sublabels, units and thresholds for these keys arrive over the wire from the catalog seed, so nothing else is needed FE-side.