briis / smartweather

WeatherFlow Smart Weather Component for Home Assistant
MIT License
110 stars 11 forks source link

Bug and Feature Request: Hour not displayed in the hourly forecast weather card #80

Closed jayscovill closed 3 years ago

jayscovill commented 3 years ago

I think this is a bug but when I select the Hourly option for the forecast the time is not displayed in the weather card, just the day. If you tap the card it correctly shows the hour.

image

Also, as a feature request would it be possible to include the option to display the daily forecast in the main card but the hourly if you tap the card for more details? Currently if you have set the hourly option it shows the hourly forecast in the details and and days for the daily.

BTW, I love your work. I use your Protect integration as well.

Cheers!

briis commented 3 years ago

BTW, I love your work. I use your Protect integration as well.

THANKS

when I select the Hourly option for the forecast the time is not displayed

Agree, this is a bug. I will look in to that.

would it be possible to include the option to display the daily forecast in the main card but the hourly if you tap the card for more details

Unfortunately that is not possible from the Weather Card, or any Custom Weather Cards I know of. I cannot built in this Integrations, as this has no Frontend options, so somebody has to built a Lovelace Weather Card, where you can supply two weather entities, one with daily data and one with hourly data.

briis commented 3 years ago

Hi Just tested this with the Built-In Weather card, and there it works without issues. Could the bug be with the Custom Card you are using?

Skærmbillede 2021-08-09 kl  16 03 47
jayscovill commented 3 years ago

You know, I could have sworn I was using the built in card but it turns out I am using someone’s custom card. It works just fine with the built in card. Sorry for bothering you about this!

I also appreciate the feedback about the feature request not being do-able.

Cheers!

From: Bjarne Riis @.> Date: Monday, August 9, 2021 at 10:05 AM To: briis/smartweather @.> Cc: Jay Scovill @.>, Author @.> Subject: Re: [briis/smartweather] Bug and Feature Request: Hour not displayed in the hourly forecast weather card (#80)

Hi Just tested this with the Built-In Weather card, and there it works without issues. Could the bug be with the Custom Card you are using? [Skærmbillede 2021-08-09 kl 16 03 47]https://user-images.githubusercontent.com/24776272/128719498-22d6a17d-423c-4eb1-a836-137e111a6091.png

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/briis/smartweather/issues/80#issuecomment-895252509, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AHJPTC2GHMP3QFRCY5AOWNTT37ODZANCNFSM5BUKWHRA. Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email.

briis commented 3 years ago

No worries 😁 I will close this now. Keep the questions coming.

jayscovill commented 3 years ago

Moreover, I discovered that the reason the custom card I was using wasn’t displaying the hourly was a Layer 8/ID10T error as I hadn’t configured the card correctly!

From: Bjarne Riis @.> Date: Monday, August 9, 2021 at 11:02 AM To: briis/smartweather @.> Cc: Jay Scovill @.>, Author @.> Subject: Re: [briis/smartweather] Bug and Feature Request: Hour not displayed in the hourly forecast weather card (#80)

No worries 😁 I will close this now. Keep the questions coming.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/briis/smartweather/issues/80#issuecomment-895299239, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AHJPTC42VEA353OQTANBD6LT37UYPANCNFSM5BUKWHRA. Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email.