Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

[BUG] Fahrenheit degrees should be expressed as integer (e.g 44, 64...) #221

Closed
lundqma opened this issue Nov 22, 2021 · 0 comments · Fixed by #226
Closed

[BUG] Fahrenheit degrees should be expressed as integer (e.g 44, 64...) #221

lundqma opened this issue Nov 22, 2021 · 0 comments · Fixed by #226
Labels
🐞 Bug Something isn't working

Comments

@lundqma
Copy link

lundqma commented Nov 22, 2021

Deployment details:

  • App version [e.g. v1.7.4]:
  • Platform [e.g. amd64, arm64, arm/v7]:
  • Docker image tag [e.g. latest, multiarch]:

Bug description:

When weather app converts from Celsius to Fahrenheit, the result should be rounded to an integer. E.g 68 rather than 68.1.

Steps to reproduce:

  1. Select "Weather" in settings
  2. Select "Fahrenheit" as Temperature unit/
  3. Go back to main page
  4. Sometimes rounding error will cause temperatures such as 44.1 or 60.1
@pawelmalak pawelmalak added the 🐞 Bug Something isn't working label Nov 22, 2021
@pawelmalak pawelmalak linked a pull request Nov 25, 2021 that will close this issue
Version 2.1.0 #226
Merged
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
🐞 Bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Version 2.1.0 pawelmalak/flame
2 participants
@pawelmalak @lundqma