Repository navigation
Tags: owid/etl
Tags
✨🤖 Metadata Diff: one level in the MDims tab, and follow grapher's co… …nfig rename Two changes, and the second is independent of the first. Collapse the MDims tab to the list. The three deep pages formed a closed loop: the author's scope decision was read only by the Review page, the Approve/Flag sign-off was read by nothing, and neither is consulted at merge — metadata ships through ETL, unlike chart-diff approvals, which gate etl chart-sync. They also carried a second review vocabulary, stored apart from the list's ticks so the two could not collide, which meant a change could read Reviewed in the list and Pending in the deep page indefinitely. What they contributed that nothing else does is kept: the PR brief is a collapsed expander on each card, and the dimension grid lives in the Blast radius section, which every card links into with the MDim preselected. The grid's leaves used to open the View diff page; they now open the view itself on this staging server, and the component takes its links from the caller, which is the only place that can know the app's routing. Links carrying ?mdim=&mode= land on the list. The brief is grouped by the ticks now — Reviewed / Not yet reviewed — so it cannot disagree with the badge. Gone with the workflow: the metadata_scope table, the non-list: review rows, and the per-chart Approve/Flag, which leaves "Look up any chart" as a diff. Net -1,127 lines. Separately: grapher renamed chart_configs.full to config. Production has migrated and a branch's staging server has not, so every query reading cc.full failed against the baseline — which is what this page was throwing before the collapse was finished. The column is resolved per environment now, so it works whichever side has migrated. Chart Diff reads the same column (chart_diff.py:926) and looks to have the same problem on master; not fixed here. Left undone, deliberately: a chart-side change has no PR brief any more (its home is the Charts list, which is separate work), and mdim_pages.py is misnamed now that it holds only the chart lookup. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
🔨 Cut a 0.3.11 release of owid-catalog supporting Python 3.12