Skip to content

feat(overview): assets by technology - #611

Merged
mandy-chessell merged 2 commits into
odpi:mainfrom
dwolfson:feat/overview-assets-by-technology
Oct 6, 2026
Merged

mandy-chessell merged 2 commits into
odpi:mainfrom
dwolfson:feat/overview-assets-by-technology

Conversation

@dwolfson

@dwolfson dwolfson commented Oct 5, 2026

Copy link
Copy Markdown
Member

New Assets by Technology panel in the Overview's Composition section (quickstart + freshstart), grouping assets by the deployedImplementationType they record.

Most assets record none (on the dev platform 4,526 of 4,986), so the buckets are exclusive and add up to the asset total: each technology, template placeholders (the unfilled ~{deployedImplementationType}~ on template elements) and No technology recorded. Assets are read in pages until an empty one; the panel shows Not measured if a page failed or the page limit was reached, never a partial chart.

Cost: one more paged read of all Assets in the summary, about +5-7 s on an uncached /api/overview/summary (about 22 s to 27 s here; cached for 60 s).

Tests: 12 new backend tests (tests/test_assets_by_technology.py, both handlers); page harness 103 checks quickstart / 97 freshstart, 0 failing; live payload verified (437 + 4,526 + 23 = 4,986).

🤖 Generated with Claude Code

New Composition panel grouping assets by the deployedImplementationType
they record. Most assets record none, so the buckets are exclusive and add
up to the asset total: each technology, template placeholders (the
unfilled ~{deployedImplementationType}~ on template elements) and "No
technology recorded". Read in pages until an empty one; None (shown "Not
measured") if a page failed or the page limit was reached.

Signed-off-by: Dan Wolfson <dan.wolfson@pdr-associates.com>
@mandy-chessell
mandy-chessell merged commit 8d874f2 into odpi:main Oct 6, 2026
5 checks passed
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