-
-
Notifications
You must be signed in to change notification settings - Fork 760
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
Remove references to old setting #8018
Remove references to old setting #8018
Conversation
- Now offloaded to plugins
✅ Deploy Preview for inventree-web-pui-preview ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #8018 +/- ##
==========================================
+ Coverage 83.79% 83.83% +0.03%
==========================================
Files 1142 1143 +1
Lines 50997 51016 +19
Branches 1786 1786
==========================================
+ Hits 42735 42770 +35
+ Misses 7815 7797 -18
- Partials 447 449 +2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
* add test for asset redirect * fully cover spa_bundler * test token api * check without beeing authed * not possible to be reached - no cover * remove unneeded except * fully test group apis * move ignore * add tests for admin site * add full admin testing * use output as ref * ignore admin edge-case * test display name settings * refactor admin test * add more admin testing * fix tests assertation * fix assertations * add test for importer admin * remove old test for function that will not be re-added for now see #8018 (comment) * Add stock detail with wrong pk * add a few stock tests
After the report generation refactor we no longer separate report generation concepts based on the "target" of the report.
This PR removes two settings which are no longer relevant.