-
-
Notifications
You must be signed in to change notification settings - Fork 289
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
Ewelink websocket #1977
Ewelink websocket #1977
Conversation
3024450
to
7495dbb
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1977 +/- ##
==========================================
+ Coverage 98.32% 98.39% +0.06%
==========================================
Files 836 860 +24
Lines 13269 13469 +200
==========================================
+ Hits 13047 13253 +206
+ Misses 222 216 -6 ☔ View full report in Codecov by Sentry. |
a095650
to
b0f7022
Compare
a76f65d
to
d72b4e5
Compare
d72b4e5
to
5e2507a
Compare
5e2507a
to
57d689a
Compare
57d689a
to
589bd11
Compare
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Pull Request check-list
To ensure your Pull Request can be accepted as fast as possible, make sure to review and check all of these items:
npm test
on both front/server)npm run eslint
on both front/server)npm run prettier
on both front/server)npm run compare-translations
on front)front/src/config/demo.js
) so that the demo website is working without a backend? (if needed) See https://demo.gladysassistant.com.NOTE: these things are not required to open a PR and can be done afterwards / while the PR is open.
Description of change
Changes eWeLink API with OAuth2 authentication.
Implements WebSocket connection as recommanded in developer documentation.
Documentation: GladysAssistant/v4-website#232