Skip to content

[Bug][Subscription Billing] Refactor Contract Line Page Loading Logic to Prevent corrupting Subscription Line#6925

Open
miljance wants to merge 1 commit intomicrosoft:mainfrom
miljance:SBRefactorContractLinePageLoadingLogic
Open

[Bug][Subscription Billing] Refactor Contract Line Page Loading Logic to Prevent corrupting Subscription Line#6925
miljance wants to merge 1 commit intomicrosoft:mainfrom
miljance:SBRefactorContractLinePageLoadingLogic

Conversation

@miljance
Copy link
Contributor

@miljance miljance commented Mar 2, 2026

Summary

This pull request refactors several pages in the Subscription Billing app to remove redundant initialization procedures and streamline how service commitment and service object records are loaded. The changes improve code consistency, reduce duplication, and enhance maintainability. Additionally, minor logic improvements were made to ensure correct handling of temporary tables and comment lines.

Work Item(s)

Fixes #6280

Fixes AB#625943

…ption, billing base period misuse on comment lines, and stale timestamp errors
@miljance miljance requested a review from a team as a code owner March 2, 2026 16:00
@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 2, 2026
@JesperSchulz JesperSchulz added the Finance GitHub request for Finance area label Mar 5, 2026
@JesperSchulz JesperSchulz added the Linked Issue is linked to a Azure Boards work item label Mar 18, 2026
@github-actions github-actions bot added this to the Version 29.0 milestone Mar 18, 2026
@JesperSchulz JesperSchulz enabled auto-merge (squash) March 19, 2026 14:27
auto-merge was automatically disabled March 19, 2026 14:39

Pull request was closed

@JesperSchulz
Copy link
Contributor

Re-triggering CI.

@JesperSchulz JesperSchulz reopened this Mar 19, 2026
@JesperSchulz JesperSchulz enabled auto-merge (squash) March 19, 2026 14:40
auto-merge was automatically disabled March 20, 2026 09:34

Pull request was closed

@djukicmilica djukicmilica reopened this Mar 20, 2026
@djukicmilica djukicmilica enabled auto-merge (squash) March 20, 2026 09:34
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]: [Subscription Billing] Refactor Contract Line Page Loading Logic to Prevent corrupting Subscription Line

3 participants