blynkkk / blynk_Issues

6 stars 6 forks source link

Units do not match datastream setting #171

Closed Selah50 closed 3 years ago

Selah50 commented 3 years ago

Issue 1: I have 2 datastreams that are set to Fahrenheit. Yesterday, they started displaying in Celsius. Issue 2: The numeric input widget that is displaying Celsius and not Fahrenheit is accepting an input that is above/below the min/max settings for its datastream when using the + and - increment buttons.

Video: https://photos.app.goo.gl/LeZde31q3W1zgcYX8

This issue is on android samsung_SM_G975F_29_2021_07_06.log

doom369 commented 3 years ago

Issue 1: I have 2 datastreams that are set to Fahrenheit. Yesterday, they started displaying in Celsius.

Could you please clarify - do you mean they show converted values in the widget or they just show the wrong suffix?

Issue 2: The numeric input widget that is displaying Celsius and not Fahrenheit is accepting an input that is above/below the min/max settings for its datastream when using the + and - increment buttons.

@alexkipar Please take a look.

alexkipar commented 3 years ago

@doom369 "unitSystem":"METRIC" in the profile.settings, and also datastreams returned by the server (in get device dashboard command) are using Celcius units

Selah50 commented 3 years ago

It appears that it is just displaying the wrong suffix.

doom369 commented 3 years ago

"unitSystem":"METRIC" in the profile.settings

@alexkipar Why do we allow that in the app? It should be hidden until it's fully implemented. The default value should be null.

are using Celcius units

Yes, because it's a metric system. Response is correct.

alexkipar commented 3 years ago

@doom369 it is definitely disabled in settings, while the value is Metric

doom369 commented 3 years ago

@alexkipar that could be set only from the mobile...

alexkipar commented 3 years ago

@doom369 it can't be set from the app

alexkipar commented 3 years ago

@Selah50 till the end of the week, we will have a small bugfix update; I suppose this issue will be fixed.

mike1964Blynk commented 3 years ago

Same issue, Template set to "Fahrenheit", Mobile Dashboard shows Celsius ("C"). will wait for the update.

Thanks, Mike

doom369 commented 3 years ago

@alexkipar have you released the update?

alexkipar commented 3 years ago

Not yet, this issue will be fixed in next build it will be available on weekend

doom369 commented 3 years ago

@Selah50 @mike1964Blynk should be fixed with the latest app. Please reopen if not fixed.

mike1964Blynk commented 3 years ago

Confirmed, Thank you, Sent from my Galaxy -------- Original message --------From: Dmitriy Dumanskiy @.> Date: 7/13/21 11:40 AM (GMT-05:00) To: blynkkk/blynk_Issues @.> Cc: mike1964Blynk @.>, Mention @.> Subject: Re: [blynkkk/blynk_Issues] Units do not match datastream setting (#171) @Selah50 @mike1964Blynk should be fixed with the latest app. Please reopen if not fixed.

—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or unsubscribe.