Skip to content

[Subcontracting] Code issue in the procedure DeleteEnhancedDocumentsByChangeOfVendorNo#8384

Open
AleksandricMarko wants to merge 1 commit into
mainfrom
bugs/main-623643-code_fix
Open

[Subcontracting] Code issue in the procedure DeleteEnhancedDocumentsByChangeOfVendorNo#8384
AleksandricMarko wants to merge 1 commit into
mainfrom
bugs/main-623643-code_fix

Conversation

@AleksandricMarko
Copy link
Copy Markdown
Contributor

@AleksandricMarko AleksandricMarko commented Jun 1, 2026

What & why

Linked work

Fixes AB#623643

How I validated this

  • I read the full diff and it contains only changes I intended.
  • I built the affected app(s) locally with no new analyzer warnings.
  • I ran the change in Business Central and confirmed it behaves as expected.
  • I added or updated tests for the new behavior, or explained below why none are needed.

What I tested and the outcome (required — be specific: scenarios, commands, screenshots for UI changes)

Risk & compatibility

@AleksandricMarko AleksandricMarko added the Subcontracting Subcontracting related activities label Jun 1, 2026
@github-actions github-actions Bot added the AL: Apps (W1) Add-on apps for W1 label Jun 1, 2026
@github-actions github-actions Bot modified the milestone: Version 29.0 Jun 1, 2026
@AleksandricMarko AleksandricMarko marked this pull request as ready for review June 2, 2026 06:56
@AleksandricMarko AleksandricMarko requested a review from a team as a code owner June 2, 2026 06:56
@AleksandricMarko AleksandricMarko enabled auto-merge (squash) June 2, 2026 06:56
@ChethanT
Copy link
Copy Markdown
Contributor

ChethanT commented Jun 2, 2026

@AleksandricMarko what issue is it and how does this fix solves it?

@AleksandricMarko
Copy link
Copy Markdown
Contributor Author

@AleksandricMarko what issue is it and how does this fix solves it?

This code is just written with wrong filters/variables, so I created PR to fix it.
We talked a month ago about this ItemLedgEntry2 and Claude also noticed wrong filter

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AL: Apps (W1) Add-on apps for W1 Subcontracting Subcontracting related activities

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants