Skip to content

Update inventory summary to use new v4 api and display more statistics#70

Open
nllong wants to merge 5 commits into
mainfrom
update-inventory-summary-page
Open

Update inventory summary to use new v4 api and display more statistics#70
nllong wants to merge 5 commits into
mainfrom
update-inventory-summary-page

Conversation

@nllong

@nllong nllong commented Jul 8, 2026

Copy link
Copy Markdown
Member
image

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the inventory summary view to use the new v4 property column-summary API and expands the UI to support selecting cycles/columns/statistics, exporting CSV, and richer per-column statistics in the grid.

Changes:

  • Replace legacy analysis-summary fetching with /api/v4/properties/column_summary/ via a new PropertyService and types.
  • Add multi-select pickers (cycles/columns/stats), grid context menu actions, and CSV export to the summary UI.
  • Update developer docs and pnpm installation/build safeguards.

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
src/app/modules/inventory-list/summary/summary.component.ts Migrates data loading to v4 API, builds dynamic stat columns, adds filtering/context menu/selection state.
src/app/modules/inventory-list/summary/summary.component.html Replaces single-cycle select with multi-pickers and adds CSV export + custom context menu UI.
src/@seed/api/property/property.types.ts Introduces response/types for v4 property column summary endpoint.
src/@seed/api/property/property.service.ts Adds PropertyService.columnSummary() wrapper for the v4 endpoint.
src/@seed/api/property/index.ts Exports new property API module.
src/@seed/api/index.ts Re-exports the property module from the API barrel.
src/@seed/styles/tailwind.scss Comment typo fix (“macOS”).
README.md Clarifies local run modes and proxy expectations; documents SEED_HOST.
pnpm-workspace.yaml Updates pnpm build-allowlist configuration for native/built deps.
package.json Adds a preinstall guard enforcing pnpm usage.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/app/modules/inventory-list/summary/summary.component.ts Outdated
Comment thread src/app/modules/inventory-list/summary/summary.component.html
Comment thread src/app/modules/inventory-list/summary/summary.component.ts Outdated
Comment thread src/app/modules/inventory-list/summary/summary.component.ts Outdated
nllong and others added 3 commits July 8, 2026 21:17
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@nllong nllong requested a review from kflemin July 9, 2026 03:28
@nllong nllong changed the title update inventory summary to use new v4 api and display more statistics Update inventory summary to use new v4 api and display more statistics Jul 9, 2026
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