Skip to content

OBPIH-6959 use ABC Class from inventory level in cycle count products list#5020

Merged
awalkowiak merged 1 commit intodevelopfrom
bug/OBPIH-6959-cycle-count-session-use-inventory-level-abcclass
Jan 29, 2025
Merged

OBPIH-6959 use ABC Class from inventory level in cycle count products list#5020
awalkowiak merged 1 commit intodevelopfrom
bug/OBPIH-6959-cycle-count-session-use-inventory-level-abcclass

Conversation

@ewaterman
Copy link
Member

✨ Description of Change

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

Description: When generating abcClass in cycle_count_session, first check if there's an inventory level defined for the product, and use abcClass from there. Otherwise fallback to the abcClass as defined in the product

@ewaterman ewaterman self-assigned this Jan 28, 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 flag: schema change Hilights a pull request that contains a change to the database schema labels Jan 28, 2025
@codecov
Copy link

codecov bot commented Jan 28, 2025

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 7.75%. Comparing base (b290f9c) to head (2999614).
Report is 201 commits behind head on develop.

Files with missing lines Patch % Lines
...g/pih/warehouse/inventory/CycleCountService.groovy 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff            @@
##             develop   #5020   +/-   ##
=========================================
  Coverage       7.75%   7.75%           
  Complexity       860     860           
=========================================
  Files            613     613           
  Lines          42623   42622    -1     
  Branches       10350   10350           
=========================================
  Hits            3306    3306           
+ Misses         38823   38822    -1     
  Partials         494     494           

☔ 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 55ecc59 into develop Jan 29, 2025
9 checks passed
@awalkowiak awalkowiak deleted the bug/OBPIH-6959-cycle-count-session-use-inventory-level-abcclass branch January 29, 2025 12:24
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 flag: schema change Hilights a pull request that contains a change to the database schema type: bug Addresses unintended behaviours of the app

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants