Skip to content

OBPIH-7514 create 0 quantity baselines when migrating empty product i…#5526

Merged
awalkowiak merged 2 commits intorelease/0.9.5-hotfix3from
bug/OBPIH-7514-migrate-empty-transactions
Oct 2, 2025
Merged

OBPIH-7514 create 0 quantity baselines when migrating empty product i…#5526
awalkowiak merged 2 commits intorelease/0.9.5-hotfix3from
bug/OBPIH-7514-migrate-empty-transactions

Conversation

@ewaterman
Copy link
Member

…nventory transactions

✨ Description of Change

Link to GitHub issue or Jira ticket: https://pihemr.atlassian.net/browse/OBPIH-7514

Description: A temp solution to https://pihemr.atlassian.net/browse/OBPIH-7230 for the hotfix release. This is mostly just a copy paste of #5517 but having it only apply to migrations, not the other features that create baselines. As the TODO says, we should revert this change when copying back up to develop.


📷 Screenshots & Recordings (optional)

Before:
Screenshot from 2025-10-01 10-07-19

After:
Screenshot from 2025-10-01 10-07-32

@ewaterman ewaterman self-assigned this Oct 1, 2025
@github-actions github-actions bot added type: bug Addresses unintended behaviours of the app domain: backend Changes or discussions relating to the backend server labels Oct 1, 2025
// 5517. It is meant to be a temporary fix to OBPIH-7514 for the 0.9.5-hotfix3 release so that we can make
// as small of a change as possible for the hotfix (we don't want to impact the other features that create
// baselines). PR 5517 is targeted for 0.9.6 and so once 0.9.5-hotfix3 is completed, we should revert this.
Transaction createBaselineTransactionForMigration(
Copy link
Member Author

Choose a reason for hiding this comment

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

As the TODO says, this is just a copy paste of ProductInventoryTransactionService.createInventoryBaselineTransactionForGivenStock with the changes from #5517 applied.

Since this is meant to be a temp change that we eventually revert when copying back up to develop, I figured this was the simplest way to do it without affecting any other code.

@awalkowiak awalkowiak merged commit 525b5b5 into release/0.9.5-hotfix3 Oct 2, 2025
3 of 5 checks passed
@awalkowiak awalkowiak deleted the bug/OBPIH-7514-migrate-empty-transactions branch October 2, 2025 12:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

domain: backend Changes or discussions relating to the backend server type: bug Addresses unintended behaviours of the app

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants