-
-
Notifications
You must be signed in to change notification settings - Fork 790
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
WIP: Websockets #7724
base: master
Are you sure you want to change the base?
WIP: Websockets #7724
Conversation
✅ Deploy Preview for inventree-web-pui-preview ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Quality Gate passedIssues Measures |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #7724 +/- ##
==========================================
+ Coverage 83.75% 84.25% +0.50%
==========================================
Files 1143 1148 +5
Lines 51084 51252 +168
Branches 1790 1794 +4
==========================================
+ Hits 42787 43184 +397
- Misses 7836 7857 +21
+ Partials 461 211 -250
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Nice that you're starting working on this, this may later can be used to stream the machine status page in real time if there are any changes one e.g. machine states, errors, ... |
It will also enable removing polling jobs fully and integrating with mqtt within plugins. Due to changes to the gunicorn config, I am a bit cautious - this should probably be targeted to 0.17.0 |
following sonar scan reports
@SchrodingersGat @wolflu05 I am facing some issues with CORS on the static icons as seen in recent tests. I can not figure out what is happening here and this is blocking the whole PR as it is failing all PUI tests. Have any of you an idea what this could be? |
Not sure, it seems that the frontend is hosted on a different port than the server for the playwright tests and cors is not allowed on the server? |
This PR is heavy WIP!! Do not run this in production
This PR adds ASGI support to the backend setup
runserver
Tasks
Closes #6262