Skip to content

[Bug][Subscription Billing] Refactor Contract Line Page Loading Logic to Prevent corrupting Subscription Line for 28.x#7486

Merged
djukicmilica merged 1 commit intomicrosoft:releases/28.xfrom
miljance:SBRefactorContractLinePageLoadingLogicFor28.x
Apr 9, 2026
Merged

[Bug][Subscription Billing] Refactor Contract Line Page Loading Logic to Prevent corrupting Subscription Line for 28.x#7486
djukicmilica merged 1 commit intomicrosoft:releases/28.xfrom
miljance:SBRefactorContractLinePageLoadingLogicFor28.x

Conversation

@miljance
Copy link
Copy Markdown
Contributor

@miljance miljance commented Mar 31, 2026 •
edited by github-actions bot
Loading

Summary

This is a backport of #6925

Work Item(s)

Fixes #6280
Fixes AB#630416

@miljance
Refactor contract line page loading logic to prevent blob field corru…
abb17e8 
…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 31, 2026 15:47
@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 31, 2026
@JesperSchulz JesperSchulz added the Finance GitHub request for Finance area label Apr 1, 2026
@djukicmilica djukicmilica enabled auto-merge (squash) April 1, 2026 13:18
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 5, 2026

⚠️ Stale Status Check Deleted

The Pull Request Build workflow run for this PR was older than 72 hours and has been deleted.

📋 Why was it deleted?

Status checks that are too old may no longer reflect the current state of the target branch. To ensure this PR is validated against the latest code and passes up-to-date checks, a fresh build is required.

🔄 How to trigger a new status check:

  1. 📤 Push a new commit to the PR branch, or
  2. 🔁 Close and reopen the PR

This will automatically trigger a new Pull Request Build workflow run.

AndersLarsenMicrosoft
AndersLarsenMicrosoft approved these changes Apr 7, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@AndersLarsenMicrosoft AndersLarsenMicrosoft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

approved

@github-actions github-actions bot added the Linked Issue is linked to a Azure Boards work item label Apr 8, 2026
@github-actions github-actions bot added this to the Version 28.1 milestone Apr 8, 2026
auto-merge was automatically disabled April 9, 2026 09:25

Pull request was closed

@djukicmilica djukicmilica reopened this Apr 9, 2026
@djukicmilica djukicmilica enabled auto-merge (squash) April 9, 2026 09:25
auto-merge was automatically disabled April 9, 2026 11:35

Pull request was closed

@djukicmilica djukicmilica reopened this Apr 9, 2026
@djukicmilica djukicmilica enabled auto-merge (squash) April 9, 2026 12:38
@djukicmilica djukicmilica merged commit 1bd2392 into microsoft:releases/28.x Apr 9, 2026
94 of 147 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@djukicmilica djukicmilica djukicmilica approved these changes

@AndersLarsenMicrosoft AndersLarsenMicrosoft AndersLarsenMicrosoft approved these changes

Assignees

@djukicmilica djukicmilica

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

Milestone

Version 28.1

Development

Successfully merging this pull request may close these issues.

4 participants

@miljance @djukicmilica @AndersLarsenMicrosoft @JesperSchulz