Skip to content

OBPIH-6887 products cant be deactivated while on a stocklist#5557

Merged
alannadolny merged 2 commits intodevelopfrom
ft/OBPIH-6887-product-cant-deactivate-when-on-stocklist
Oct 17, 2025
Merged

OBPIH-6887 products cant be deactivated while on a stocklist#5557
alannadolny merged 2 commits intodevelopfrom
ft/OBPIH-6887-product-cant-deactivate-when-on-stocklist

Conversation

@ewaterman
Copy link
Member

✨ Description of Change

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

Description: Don't allow a product to be deactivated if it is a part of an active stocklist


📷 Screenshots & Recordings (optional)

Screenshot from 2025-10-16 09-46-57

@ewaterman ewaterman self-assigned this Oct 16, 2025
@github-actions github-actions bot added type: feature A new piece of functionality for the app domain: backend Changes or discussions relating to the backend server domain: l10n Changes or discussions relating to localization & Internationalization labels Oct 16, 2025
@codecov
Copy link

codecov bot commented Oct 16, 2025

Codecov Report

❌ Patch coverage is 15.38462% with 11 lines in your changes missing coverage. Please review.
✅ Project coverage is 8.49%. Comparing base (1bb7314) to head (809d089).
⚠️ Report is 146 commits behind head on develop.

Files with missing lines Patch % Lines
...pp/domain/org/pih/warehouse/product/Product.groovy 15.38% 10 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             develop   #5557      +/-   ##
============================================
- Coverage       9.12%   8.49%   -0.63%     
+ Complexity      1170    1106      -64     
============================================
  Files            701     701              
  Lines          45281   45277       -4     
  Branches       10851   10850       -1     
============================================
- Hits            4131    3847     -284     
- Misses         40497   40857     +360     
+ Partials         653     573      -80     

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

Comment on lines 20 to 21
import org.pih.warehouse.api.Stocklist
import org.pih.warehouse.api.StocklistItem
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do we need the Stocklist and StocklistItem imports? They don’t seem to be used here, but I might be wrong 🤔

@alannadolny alannadolny merged commit 44736ed into develop Oct 17, 2025
7 checks passed
@alannadolny alannadolny deleted the ft/OBPIH-6887-product-cant-deactivate-when-on-stocklist branch October 17, 2025 08:46
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 domain: l10n Changes or discussions relating to localization & Internationalization type: feature A new piece of functionality for the app

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants