-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Proposal: Show Rain Delay status on the top-level card #16
Comments
Alrighty, I've got everything working to dynamically filter for the
Yes, good point here... I'll add it to the
I like that! So, further questions for you, with apologies that I'm a dangerous newbie at best in JavaScript 😄: I looked briefly at implementing this, and think there are a few possible paths depending on your philosophy about dependencies... what would you recommend/prefer as an approach? In general I will need to:
A few options:
Any preferences or recommendations there? Left to my own devices I would probably try (2) with My other question on any of the above is whether it's safe to rely on browser support for Thanks for your thoughts & time! 😁 |
Quick update: In further testing, I found a bug in the underlying python library that powers the HACS integration. The Rain Delay Stop Time is not actually corrected for OpenSprinkler's time zone settings, resulting in a local timestamp being incorrectly labeled as UTC in home assistant, and badness ensuing when computing a relative time. I missed this when doing simple date formatting since I wasn't pretty printing the timezone... Working on fixing it quick, then will circle back to this PR since the relative time math really depends on valid underlying timestamps 😄 |
OK, got the changes to the upstream python library used in the integration merged, so correct timestamp data will be flowing for the rain delay stop time. I researched the relative time a bit more, and found that Let's use that. Details in the PR #17:
|
👋 Hi! New to OpenSprinkler, the HACS integration, and this Lovelace card...
At the moment I'm not yet comfortable with how the automatic weather handling algorithms might interact with new seeds in my garden, so I'm managing weather manually with the OpenSprinkler rain delay functionality.
I want to propose the idea of showing the rain delay status on the top-level card while active... similar to the brief banner across the top that you see on the OpenSprinkler home page or mobile app.
I've got a prototype that comes out looking like this... but before I pitch that as a PR, wanted to get your reaction as to whether that's a direction you would be interested in going or not:
Thoughts?
Basically looking to extend the high-level view of what's happening with my OpenSprinkler to also include "nothing... because there's a rain delay."
I'm interested in how to actually update the rain delay from Home Assistant as well, but need to dig into whether that's exposed by the API and integration or not. If so, perhaps it can be exposed through the card similar to the manual station time with the
input_number
support.The text was updated successfully, but these errors were encountered: