Skip to content
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

Notification: "when_relative" for "chronometer" #4273

Merged
merged 1 commit into from
Mar 22, 2024

Conversation

Roffild
Copy link
Contributor

@Roffild Roffild commented Mar 15, 2024

"timeout" is relative, but "when" is absolute by default.

It is now possible to synchronize timers in seconds without time difference:

timeout: 120
chronometer: "true"
when: 120
when_relative: "true"

"timeout" is relative, but "when" is absolute by default.

It is now possible to synchronize timers in seconds without time difference:
```yaml
timeout: 120
chronometer: "true"
when: 120
when_relative: "true"
```
@dshokouhi
Copy link
Member

This PR requires the new parameter to be added to the server side code as well.

https://github.com/home-assistant/mobile-apps-fcm-push

Copy link
Member

@dshokouhi dshokouhi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please attach a PR to the FCM repo adding the new parameter and link it here, otherwise Firebase notifications will be missing this parameter completely. You may refer to a past PR in that repo to see how to add it.

@home-assistant home-assistant bot marked this pull request as draft March 15, 2024 14:53
@home-assistant
Copy link

Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍

Learn more about our pull request process.

Copy link
Member

@dshokouhi dshokouhi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, nice to sync up the chronometer countdown with the timeout feature :)

@jpelgrom jpelgrom merged commit 5729bbe into home-assistant:master Mar 22, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants