-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
refactor(backend): reduce surface by unifying metadata endpoints #11035
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
base: master
Are you sure you want to change the base?
Conversation
…t of permanent redirects
✅ Deploy Preview for inventree-web-pui-preview canceled.
|
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## master #11035 +/- ##
==========================================
- Coverage 88.18% 88.16% -0.03%
==========================================
Files 1290 1290
Lines 58142 58145 +3
Branches 1969 1969
==========================================
- Hits 51273 51264 -9
- Misses 6378 6390 +12
Partials 491 491
Flags with carried forward coverage won't be shown. Click here to find out more.
🚀 New features to boost your workflow:
|
…reduce-metadata-endpoints
…reduce-metadata-endpoints
…r/InvenTree into reduce-metadata-endpoints
…reduce-metadata-endpoints
…reduce-metadata-endpoints
…reduce-metadata-endpoints
CodSpeed Performance ReportMerging #11035 will not alter performanceComparing Summary
Footnotes |
|
@SchrodingersGat this is ready for review, the needed change for the API client is already proposed inventree/inventree-python#285 You can see the passing test with that branch in https://github.com/inventree/InvenTree/actions/runs/20733404036/job/59525935718 |
|
@matmair this does look a lot cleaner. To confirm what you are doing here - all metadata is accessed via |
Replaces individual metadata endpoints with a generic endpoint and a bunch of permanent redirects. Obviously a breaking change.
This:
Requires a change to the client libary - inventree/inventree-python#285