Skip to content

[Bug][SubscriptionBilling]: Printing multiple posted sales invoices from subscription contracts fails with duplicate record error#7128

Merged
djukicmilica merged 2 commits intomicrosoft:mainfrom
miljance:SBInvoiceBatchPrint
Mar 26, 2026
Merged

[Bug][SubscriptionBilling]: Printing multiple posted sales invoices from subscription contracts fails with duplicate record error#7128
djukicmilica merged 2 commits intomicrosoft:mainfrom
miljance:SBInvoiceBatchPrint

Conversation

@miljance
Copy link
Copy Markdown
Contributor

@miljance miljance commented Mar 13, 2026

Summary

This pull request introduces improvements to buffer management in the Subscription Billing module. The main focus is on ensuring that temporary buffers are properly reset and cleared before use, which helps prevent data inconsistencies and improves reliability.

Work Item(s)

Fixes #6973

Fixes AB#625923

@miljance miljance requested a review from a team as a code owner March 13, 2026 15:11
@github-actions github-actions Bot added AL: Apps (W1) Add-on apps for W1 From Fork Pull request is coming from a fork labels Mar 13, 2026
@JesperSchulz JesperSchulz added Finance GitHub request for Finance area Linked Issue is linked to a Azure Boards work item labels Mar 16, 2026
@github-actions github-actions Bot added this to the Version 29.0 milestone Mar 18, 2026
auto-merge was automatically disabled March 20, 2026 09:38

Pull request was closed

@djukicmilica djukicmilica reopened this Mar 20, 2026
@djukicmilica djukicmilica enabled auto-merge (squash) March 20, 2026 09:38
@miljance
Copy link
Copy Markdown
Contributor Author

@JesperSchulz and @djukicmilica the test should no longer fail. The test was introduced on 18.3.2026 and then disabled on 20.3.2026.

image

auto-merge was automatically disabled March 23, 2026 16:15

Pull request was closed

@djukicmilica djukicmilica reopened this Mar 23, 2026
@djukicmilica djukicmilica merged commit b378c97 into microsoft:main Mar 26, 2026
40 of 44 checks passed
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 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.

[Bug][SubscriptionBilling]: Printing multiple posted sales invoices from subscription contracts fails with duplicate record error

4 participants