mikael
September 9, 2021, 6:36pm
1
Hello,
I would like to know how to change °F to °C with Telegram when responding to Gladys.
Thanks in advance,
Go to « Settings » => « Users » and then you can edit your user and set the temperature unit you want
Note: I know there was a bug about this a few versions ago in Gladys (Fahrenheit was used instead of Celsius), check that you are on the latest version of Gladys (v4.5.1), you can check in « Settings » => « System »
mikael
September 10, 2021, 11:40am
3
The user is indeed in Celsius and I am indeed in 4.5.1.
Which phrases work and which ones return °F?
I would like the exact phrase, it must be a bug
mikael
September 10, 2021, 12:11pm
5
What is the temperature in the living room?
What temperature is it in the room?
Oh yes, and if I write: « la? » I get the weather outside
Ok, noted!
I created a bug report on GitHub, I’ll check and get back to you
https://github.com/GladysAssistant/Gladys/issues/1293
@mikael For info, I tried to work on this bug, I can’t reproduce it, everything works on my side:
Can you confirm that in your profile, you have set your account to « Celsius »?
Don’t hesitate to change/then switch back to Celsius to be sure
mikael
November 12, 2021, 8:26am
8
In the chat box, when I ask for the temperature of a room it shows in Celsius, but when I ask via Telegram it shows in Fahrenheit.
I had already made the change in the profile.
I just fixed the bug here:
master ← fix-telegram-temperature-unit-bug
ouvert 08:51AM - 12 Nov 21 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:
- [ ] If your changes affects code, did your write the tests?
- [ ] Are tests passing? (`npm test` on both front/server)
- [ ] Is the linter passing? (`npm run eslint` on both front/server)
- [ ] Did you run prettier? (`npm run prettier` on both front/server)
- [ ] If you are adding a new features/services, did you run integration comparator? (`npm run compare-translations` on front)
- [ ] 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.
- [ ] If your changes modify the API (REST or Node.js), did you modify the API documentation? (Documentation is based on comments in code)
- [ ] 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).
- [ ] 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!
