Skip to content

OBPIH-7196 switch product merge to use baseline transaction type#5258

Merged
awalkowiak merged 1 commit intodevelopfrom
maintenance/OBPIH-7196-product-merge-transaction-type
May 16, 2025
Merged

OBPIH-7196 switch product merge to use baseline transaction type#5258
awalkowiak merged 1 commit intodevelopfrom
maintenance/OBPIH-7196-product-merge-transaction-type

Conversation

@ewaterman
Copy link
Member

✨ Description of Change

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

Description: Switch the product merge feature to use the new inventory baseline transaction type.

@ewaterman ewaterman requested a review from awalkowiak May 16, 2025 15:10
@ewaterman ewaterman self-assigned this May 16, 2025
@github-actions github-actions bot added type: maintenance Code improvements, optimizations and refactors, dependency upgrades... domain: backend Changes or discussions relating to the backend server labels May 16, 2025
Transaction transaction = new Transaction()
transaction.inventory = inventory
transaction.transactionType = TransactionType.get(Constants.PRODUCT_INVENTORY_TRANSACTION_TYPE_ID)
transaction.transactionType = TransactionType.get(Constants.INVENTORY_BASELINE_TRANSACTION_TYPE_ID)
Copy link
Member Author

Choose a reason for hiding this comment

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

it feels like it shouldn't be this simple 😅 I'm looking through to code to make sure I didn't miss something but let me know if you have any thoughts since I know you created the feature.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Nope, that's it. But we included some time for QA to check this one.

@codecov
Copy link

codecov bot commented May 16, 2025

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 8.27%. Comparing base (352f7a1) to head (1e5368e).
⚠️ Report is 137 commits behind head on develop.

Files with missing lines Patch % Lines
...g/pih/warehouse/product/ProductMergeService.groovy 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff            @@
##             develop   #5258   +/-   ##
=========================================
  Coverage       8.27%   8.27%           
  Complexity       965     965           
=========================================
  Files            641     641           
  Lines          43288   43288           
  Branches       10520   10520           
=========================================
  Hits            3583    3583           
  Misses         39155   39155           
  Partials         550     550           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@awalkowiak awalkowiak merged commit 97c81e8 into develop May 16, 2025
9 checks passed
@awalkowiak awalkowiak deleted the maintenance/OBPIH-7196-product-merge-transaction-type branch May 16, 2025 15:20
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: maintenance Code improvements, optimizations and refactors, dependency upgrades...

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants