Skip to content

Try to add microsoftInternal__NAV-210537 - #15

Closed
haoranpb wants to merge 1 commit into
mainfrom
dataset/nav-210537
Closed

Try to add microsoftInternal__NAV-210537#15
haoranpb wants to merge 1 commit into
mainfrom
dataset/nav-210537

Conversation

@haoranpb

@haoranpb haoranpb commented Oct 21, 2025

Copy link
Copy Markdown
Collaborator

@haoranpb haoranpb closed this Oct 21, 2025
@haoranpb
haoranpb deleted the dataset/nav-210537 branch October 21, 2025 15:45
onbuyuka pushed a commit that referenced this pull request Jul 28, 2026
AL query Count columns take no source field: `column(RowCount) { Method = Count; }`,
not `column(RowCount; "No.") { Method = Count; }` (the latter fails AL0353). The
four Count-based golds used the invalid form, and SKILL.md taught it — so the agent
reproduced the mistake and its query failed to compile before the gold was ever
reached, which is why these golds went unvalidated (see PR review comment #15).

- Remove the source field from the Count columns in customer-count-by-country,
  open-sales-order-count-by-customer, opportunity-count-by-status, and
  line-count-per-open-sales-order gold queries.
- SKILL.md: clarify that Count takes no source field, unlike Sum/Average/Min/Max.

Validated by the runner shakeout: the Sum-based new golds (outstanding-purchase-value
-by-vendor, total-purchased-quantity-by-item) already compiled, ran, and resolved.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: db34a2a0-7035-4361-b911-becb72f86e21
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