lovelace-hourly-weather icon indicating copy to clipboard operation
lovelace-hourly-weather copied to clipboard

Published 20 hours ago • verified publisher icon decompil3d

Feature: Add option to show current weather as the first segment

Open gonzalezjj opened this issue 2 years ago • 2 comments
trafficstars

This PR adds a config option to add the current weather as the first segment.:

image

image

In its current state, the PR is meant as a request for comments. The following should completed before merging:

  • Tests (I might be able to do that myself)
  • Translations (not sure how to do that, other than by using online translation services)

I'd appreciate some input about this change. Thanks!

gonzalezjj avatar Aug 23 '23 12:08 gonzalezjj

Thanks for the contribution! This looks like a good optional feature. Go ahead and add tests. For localization, the CI will automatically add the other languages when this merges into a branch in the repo itself. Let me know when you're ready for review and I can open a branch and merge yours into it before it goes to main and it should pick up the translations there.

decompil3d avatar Aug 23 '23 16:08 decompil3d

Apologies for the long delay on this. If you're still keen on adding this feature, would you mind rebasing against the latest main and getting things back up to date? Then I can merge this into a second branch so translations get picked up.

decompil3d avatar Feb 23 '24 21:02 decompil3d