Skip to content

OBPIH-7117 duplicate items error should also show on lot number field#5153

Merged
ewaterman merged 1 commit intodevelopfrom
bug/OBPIH-7117
Mar 21, 2025
Merged

OBPIH-7117 duplicate items error should also show on lot number field#5153
ewaterman merged 1 commit intodevelopfrom
bug/OBPIH-7117

Conversation

@ewaterman
Copy link
Member

✨ Description of Change

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

Description: When one or more custom rows have the same [bin + lot] as another row, display an error on both the bin and lot fields for that row. Previously we were just displaying the error on the bin field, which isn't present for facilities with bin disabled, and so the error wasn't visible.


📷 Screenshots & Recordings (optional)

For facility with bin location enabled:
Screenshot from 2025-03-20 10-15-30

For facility with bin location disabled:
Screenshot from 2025-03-20 10-13-16

@ewaterman ewaterman self-assigned this Mar 20, 2025
@github-actions github-actions bot added type: bug Addresses unintended behaviours of the app domain: frontend Changes or discussions relating to the frontend UI labels Mar 20, 2025
@codecov
Copy link

codecov bot commented Mar 20, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 8.11%. Comparing base (87f0120) to head (d988419).
Report is 153 commits behind head on develop.

Additional details and impacted files
@@            Coverage Diff            @@
##             develop   #5153   +/-   ##
=========================================
  Coverage       8.11%   8.11%           
  Complexity       937     937           
=========================================
  Files            637     637           
  Lines          43137   43137           
  Branches       10488   10488           
=========================================
  Hits            3500    3500           
  Misses         39100   39100           
  Partials         537     537           

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

@ewaterman ewaterman merged commit e4e6cf5 into develop Mar 21, 2025
9 checks passed
@ewaterman ewaterman deleted the bug/OBPIH-7117 branch March 21, 2025 18:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

domain: frontend Changes or discussions relating to the frontend UI type: bug Addresses unintended behaviours of the app

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants