Hello everyone!
I’ve almost finished the MQTT developments, the new API where MQTT device control is now coded. For now, it’s in a branch and I’d like your feedback
Here is the specification of the new API:
A small demo video of a Device → Gladys value send:
https://streamable.com/audph
But it is also possible to send data from Gladys → Device, by setting the feature as not « read-only ». Gladys will send a message on the specified topic.
The PR is available here:
master ← implement-new-mqtt-api
ouvert 02:56PM - 09 Mar 20 UTC
### Pull Request check-list
To ensure your Pull Request can be accepted as fa… st as possible, make sure to review and check all of these items:
- [x] If your changes affects code, did your write the tests?
- [x] Are tests passing? (`npm test` on both front/server)
- [x] Is the linter passing? (`npm run eslint` on both front/server)
- [x] Did you run prettier? (`npm run prettier` on both front/server)
- [x] If your changes modify the API (REST or Node.js), did you modify the API documentation? (Documentation is based on comments in code)
- [x] If you are adding a new features/services which needs explanation, did you modify the user documentation? See [the GitHub repo](https://github.com/GladysAssistant/gladys-4-docs) and the [website](https://documentation.gladysassistant.com).
- [x] Did you add fake requests data for the demo mode (`front/src/config/demo.json`) so that the demo website is working without a backend? (if needed) See [https://demo.gladysassistant.com](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
Please provide a description of the change here. It's always best with screenshots, so don't hesitate to add some!
Feel free if you have any feedback