-
-
Notifications
You must be signed in to change notification settings - Fork 821
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Barcode scanning #8732
Barcode scanning #8732
Conversation
- Separated widgets for camera / keyboard / wedge scanner - UI / UX improvements
✅ Deploy Preview for inventree-web-pui-preview ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #8732 +/- ##
==========================================
+ Coverage 85.18% 85.24% +0.05%
==========================================
Files 1173 1174 +1
Lines 51565 51548 -17
Branches 2094 2076 -18
==========================================
+ Hits 43926 43942 +16
+ Misses 7131 7084 -47
- Partials 508 522 +14
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
- Access as part of useTable hook - No longer hard-coded to 'pk'
- Simplify - Prevent errant keycodes from closing the scanner dialog
@sur5r @saschaludwig this should fix your scanning issues |
Great improvement @SchrodingersGat! Looking very nice |
Overview
This PR implements the "quick scan" dialog functionality in the new user interface, as well as refactoring / further features for the "barcode actions" page.
The "scan barcode" dialog has now been refactored into a common component, with support for:
Related Issues
TODO
Future Work