Être capable d'injecter une variable dans requête HTTP + contrôler un appareil

_Will_71 · Mai 23, 2022, 10:32

Ce serait bien aussi de pouvoir corriger le problème de variables dans les requêtes

@cicoub13 a proposé une PR mais je ne sais pas si il a fini ces tests

github.com/GladysAssistant/Gladys

fix: Sort variables to avoid bad replacement in text

GladysAssistant:master ← cicoub13:fix-1503

ouvert 05:43AM - 15 Apr 22 UTC

cicoub13

+2 -1

Fix #1503 ### Pull Request check-list To ensure your Pull Request can be a…ccepted as fast 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 you are adding a new features/services, did you run integration comparator? (`npm run compare-translations` on front) - [x] Did you test this pull request in real life? With real devices? If this development is a big feature or a new service, we recommend that you provide a Docker image to [the community](https://community.gladysassistant.com/) for testing before merging. - [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/v4-website) and the [website](https://gladysassistant.com). - [x] Did you add fake requests data for the demo mode (`front/src/config/demo.js`) 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!