-
-
Notifications
You must be signed in to change notification settings - Fork 795
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
Refactoring / Enhancements #8307
Refactoring / Enhancements #8307
Conversation
- And serializers - Basic, needs lots of work still
✅ 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 #8307 +/- ##
==========================================
- Coverage 84.35% 84.23% -0.12%
==========================================
Files 1165 1166 +1
Lines 52845 52865 +20
Branches 1931 1939 +8
==========================================
- Hits 44577 44531 -46
- Misses 7774 7850 +76
+ Partials 494 484 -10
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
- Do not request token if other cookies are unavailable - Do not fetch user data if token is unavailable - Prevents connection error logs
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.
Lots of small things, seems reasonable
WIP PR to provide historical salse information (broken down by month for e.g.) for a particular part.Note: The original intent of this PR is probably better served as a plugin. But there are a number of fixes / improvements which are still good additions.