-
-
Notifications
You must be signed in to change notification settings - Fork 773
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
Batch code generation #7000
Batch code generation #7000
Conversation
- Provide additional kwargs to plugin - Move into new file - Error handling
- Build up a dataset and query server whenever it changes - Look for result in response data - For now, just used for generating batch codes - may be used for more in the future
✅ 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 #7000 +/- ##
========================================
Coverage 84.03% 84.03%
========================================
Files 1054 1056 +2
Lines 46297 46443 +146
Branches 1326 1334 +8
========================================
+ Hits 38905 39030 +125
- Misses 7063 7077 +14
- Partials 329 336 +7
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM; it seems like the API schema changed though - could you bump the API version?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Co-authored-by: Lukas <76838159+wolflu05@users.noreply.github.com>
…venTree into batch-code-generation
Bumping this to the next release cycle, still some work to do here. |
- Move to /generate/ API endpoint - Move batch code generator too
This PR implements a solution for #6098, namely allowing batch codes to be generated via a
POST
request to an API endpoint.Any data provided to the request is made available to plugins, so a custom plugin could theoretically use this information for the purpose of batch code generation.
This also provides a solution for #6364 (at least in the new react interface), by adding a new
useGenerator
hook function that makes use of this new API endpoint.Related Issues
Tasks
Further Work
Further work could extend on this by:
generators for serial numbers and internal part numbers would hook into existing plugins, allowing these plugins to seamlessly inject data into the front-end forms dynamically