Skip to content

enhance weather panel#352

Merged
balloob merged 4 commits intohome-assistant:masterfrom
fanthos:weatherpanel
Jul 24, 2017
Merged

enhance weather panel#352
balloob merged 4 commits intohome-assistant:masterfrom
fanthos:weatherpanel

Conversation

@fanthos
Copy link
Copy Markdown
Contributor

@fanthos fanthos commented Jul 23, 2017

Make a skin for weather panel.
Multi day forecast shows in table instead of Chart.
Use Emoji for forecast weather condiction.

Using old Yahoo Weather component:

default

Reworked Yahoo Weather component:
2

fanthos added 2 commits July 23, 2017 17:17
Make a skin for weather panel.
Multi day forecast shows in table using Emoji.
@mention-bot
Copy link
Copy Markdown

@fanthos, thanks for your PR! By analyzing the history of the files in this pull request, we identified @armills, @Tommatheussen and @andrey-git to be potential reviewers.

@balloob
Copy link
Copy Markdown
Member

balloob commented Jul 24, 2017

I like it. Looks cool.

@balloob balloob merged commit 0c42813 into home-assistant:master Jul 24, 2017
@abmantis
Copy link
Copy Markdown
Member

abmantis commented Aug 3, 2017

This has messed up the OpenWeatherMap component. Previously it would show a nice 7 day graph with temperature changes, now it shows this:
screenshot from 2017-08-04 00-00-37

@mjj4791
Copy link
Copy Markdown

mjj4791 commented Aug 6, 2017

This changes also impacted the buienradar weather component

Now have a working fix for that (with some additons/new features)... (#375)
Now discover we are reverting this change?

@Landrash Landrash mentioned this pull request Aug 9, 2017
tkdrob pushed a commit to tkdrob/frontend that referenced this pull request Apr 20, 2021
* Add info about dismissed notification events

* Mention all notification data is included and not just the message

* Tweak

* One more tweak
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants