Skip to content

OBPIH-7195 fix: error creating baseline if a transaction already exists for given date#5287

Merged
awalkowiak merged 1 commit intodevelopfrom
maintenance/OBPIH-7195-1-transactions-same-time
May 30, 2025
Merged

OBPIH-7195 fix: error creating baseline if a transaction already exists for given date#5287
awalkowiak merged 1 commit intodevelopfrom
maintenance/OBPIH-7195-1-transactions-same-time

Conversation

@ewaterman
Copy link
Member

✨ Description of Change

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

Description: Two fixes:

  1. Error creating a baseline transaction if a transaction already exists for the given transaction date
  2. Exclude any items where QoH == 0 from the baseline transaction

📷 Screenshots & Recordings (optional)

Screenshot from 2025-05-28 09-38-14

@ewaterman ewaterman self-assigned this May 28, 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 28, 2025
@codecov
Copy link

codecov bot commented May 28, 2025

Codecov Report

❌ Patch coverage is 28.57143% with 15 lines in your changes missing coverage. Please review.
✅ Project coverage is 8.31%. Comparing base (a7a7aa8) to head (ac59304).
⚠️ Report is 115 commits behind head on develop.

Files with missing lines Patch % Lines
...rg/pih/warehouse/inventory/InventoryService.groovy 0.00% 14 Missing ⚠️
...ehouse/inventory/ProductAvailabilityService.groovy 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             develop   #5287      +/-   ##
============================================
- Coverage       8.34%   8.31%   -0.03%     
+ Complexity       995     991       -4     
============================================
  Files            648     648              
  Lines          43511   43530      +19     
  Branches       10549   10550       +1     
============================================
- Hits            3632    3621      -11     
- Misses         39317   39350      +33     
+ Partials         562     559       -3     

☔ 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.

Copy link
Collaborator

@awalkowiak awalkowiak left a comment

Choose a reason for hiding this comment

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

Overall looks good, I am only wondering if it would be enough to just have a validation against Inventory Baseline transaction, not all.

@awalkowiak awalkowiak merged commit cc85ea8 into develop May 30, 2025
8 checks passed
@awalkowiak awalkowiak deleted the maintenance/OBPIH-7195-1-transactions-same-time branch May 30, 2025 08:56
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.

3 participants