Skip to content

[main]-Multiple error messages when posting to period with VAT Return already in status Released#9485

Merged
MSNehaNawal merged 5 commits into
microsoft:mainfrom
MSNehaNawal:bugs/Bug-639895-main-MultipleErrormsgs_VATReturnReleased
Jul 17, 2026
Merged

[main]-Multiple error messages when posting to period with VAT Return already in status Released#9485
MSNehaNawal merged 5 commits into
microsoft:mainfrom
MSNehaNawal:bugs/Bug-639895-main-MultipleErrormsgs_VATReturnReleased

Conversation

@MSNehaNawal

@MSNehaNawal MSNehaNawal commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Bug 641514: [master] [all-e]Multiple error messages when posting to period with VAT Return already in status Released

Fixes AB#641514

@github-actions github-actions Bot added From Fork Pull request is coming from a fork Linked Issue is linked to a Azure Boards work item labels Jul 15, 2026
@github-actions github-actions Bot added this to the Version 29.0 milestone Jul 15, 2026
@JesperSchulz JesperSchulz added the Finance GitHub request for Finance area label Jul 16, 2026
@MSNehaNawal
MSNehaNawal marked this pull request as ready for review July 17, 2026 04:28
@MSNehaNawal
MSNehaNawal requested a review from a team July 17, 2026 04:28
Comment thread src/Layers/W1/Tests/VAT/ERMVATSalesPurchase.Codeunit.al
@github-actions

Copy link
Copy Markdown
Contributor

Copilot PR Review

Iteration 1 · Outcome: completed

All 16 sub-skills ran to completion; 10 returned not-applicable (no matching object types in this diff: al-upgrade-review, al-ui-review, al-web-services-review, al-data-modeling-review, al-query-review, al-appsource-review, al-telemetry-review) or completed with no findings (al-security-review, al-privacy-review, al-interfaces-review, al-style-review, al-error-handling-review). al-performance-review, al-events-review, al-breaking-changes-review, and al-testing-review each surfaced findings. The self-review cross-cutting pass found no additional candidate clearing the agent-finding precision bar.

Knowledge source: https://github.com/microsoft/BCQuality@186d8a131465475c79244d994acb872cd5c0d4bf

Findings by domain

Findings split into Knowledge-backed (cite a BCQuality article) and Agent (the agent's own judgement, no matching BCQuality rule).

Domain Findings Knowledge-backed Agent Inline Fallback
Breaking Changes 1 1 0 1 0
Events 1 1 0 1 0
Performance 1 1 0 1 0
Testing 2 1 1 2 0

Totals: 4 knowledge-backed · 1 agent findings.

Orchestrator pre-filter (2 file(s) excluded)

  • layer-disabled (knowledge) : 2 file(s)

Findings produced by the AL review agent v1.7.3. Reply 👎 on any inline comment to flag false positives.

@MSNehaNawal
MSNehaNawal requested a review from a team July 17, 2026 07:04
@MSNehaNawal
MSNehaNawal added this pull request to the merge queue Jul 17, 2026
Merged via the queue into microsoft:main with commit a5c55bf Jul 17, 2026
325 of 329 checks passed
@MSNehaNawal
MSNehaNawal deleted the bugs/Bug-639895-main-MultipleErrormsgs_VATReturnReleased branch July 17, 2026 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Finance GitHub request for Finance area From Fork Pull request is coming from a fork Linked Issue is linked to a Azure Boards work item

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants