Skip to content

[fork-CI] [20.0][IMP] sdpo: avg order value KPI + Product/Category filters - #11

Open
dnplkndll wants to merge 2 commits into
20.0-sty-sdpofrom
20.0-imp-sdpo-features
Open

dnplkndll wants to merge 2 commits into
20.0-sty-sdpofrom
20.0-imp-sdpo-features

Conversation

@dnplkndll

Copy link
Copy Markdown

Stacked on the styling branch (20.0-sty-sdpo), so the diff here is only the two additive features — this is what would go upstream as a follow-up [IMP] once the MIG lands.

  • Avg order value KPI, computed from figures already on the Data sheet (untaxed total / orders); fills the 3x2 KPI grid the styling change leaves half-empty.
  • Product and Product Category global filters, wired to all 6 pivots, the list and the chart. purchase.report carries product_id/category_id; the list reaches them through order_line.

Neither is 20.0-forced and neither was asked for in review, which is why they are not in the migration.

Verified on a clean 20.0 with demo data: filtering Product Category to Services zeroes every KPI and empties the tables, while the chart (now nested in a carousel's chartDefinitions) correctly falls to the single Services purchase — the negative control that proves the wiring rather than assuming it.

AI-assisted (Claude Code); every change reviewed, tested, and owned by the author.

Assisted-by: Claude Opus 5
… filters

Adds an average order value KPI computed from the Data sheet, plus Product and
Product Category global filters wired to the pivots, the list and the chart.

Assisted-by: Claude Opus 5
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.

1 participant