Skip to content

OBPIH-7143 fail starting count when in pending outbound#5305

Merged
awalkowiak merged 1 commit intodevelopfrom
ft/OBPIH-7143-fail-start-count-when-pending
Jun 4, 2025
Merged

OBPIH-7143 fail starting count when in pending outbound#5305
awalkowiak merged 1 commit intodevelopfrom
ft/OBPIH-7143-fail-start-count-when-pending

Conversation

@ewaterman
Copy link
Member

✨ Description of Change

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

Description: Kasia had a good point that when the product is in a pending outbound, we should also block users from starting a count (the other PR was for erroring when requesting a count). However we should only block the count from being started while it's in the "to count" state. Once there is progress in the count, it should be able to finish.

Initial PR: #5276

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

codecov bot commented Jun 3, 2025

Codecov Report

❌ Patch coverage is 0% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 8.34%. Comparing base (9aeb5c3) to head (ab02d34).
⚠️ Report is 102 commits behind head on develop.

Files with missing lines Patch % Lines
.../warehouse/inventory/CycleCountStartCommand.groovy 0.00% 7 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             develop   #5305      +/-   ##
============================================
+ Coverage       8.30%   8.34%   +0.03%     
- Complexity       994    1002       +8     
============================================
  Files            649     649              
  Lines          43650   43657       +7     
  Branches       10579   10583       +4     
============================================
+ Hits            3625    3642      +17     
+ Misses         39465   39453      -12     
- Partials         560     562       +2     

☔ 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 bcd09d1 into develop Jun 4, 2025
8 checks passed
@awalkowiak awalkowiak deleted the ft/OBPIH-7143-fail-start-count-when-pending branch June 4, 2025 08:41
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.

2 participants