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

Add ability to name timers / scheduled jobs #2911

Merged
merged 1 commit into from
Jun 18, 2022

Conversation

J-N-K
Copy link
Member

@J-N-K J-N-K commented Apr 12, 2022

Closes #1255
Closes #2750

Potentially API breaking, although I think that alternative scheduler implementations are unlikely. I could add a default for the new method that just throws away the identifier.

Signed-off-by: Jan N. Klug github@klug.nrw

@J-N-K J-N-K requested a review from a team as a code owner April 12, 2022 18:07
@wborn wborn added the enhancement An enhancement or new feature of the Core label Apr 22, 2022
Signed-off-by: Jan N. Klug <github@klug.nrw>
@J-N-K J-N-K force-pushed the feature-namedtimers branch from 3b84992 to bdea9cc Compare May 26, 2022 08:56
Copy link
Member

@wborn wborn left a comment

Choose a reason for hiding this comment

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

Seems like a very useful addition! 🙂

@wborn wborn merged commit c2ec97c into openhab:main Jun 18, 2022
@wborn wborn added this to the 3.3 milestone Jun 18, 2022
@J-N-K J-N-K deleted the feature-namedtimers branch June 18, 2022 13:06
@openhab-bot
Copy link
Collaborator

This pull request has been mentioned on openHAB Community. There might be relevant details there:

https://community.openhab.org/t/way-to-name-timers-in-js-scripting/138156/1

florian-h05 added a commit to florian-h05/openhab-js that referenced this pull request Sep 13, 2022
Fixes openhab#159.
Refers to openhab/openhab-core#2911.

Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
florian-h05 added a commit to openhab/openhab-js that referenced this pull request Sep 13, 2022
Fixes #159.
Refers to openhab/openhab-core#2911.

Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
florian-h05 added a commit to florian-h05/openhab-rules-tools that referenced this pull request Sep 28, 2022
This change names the created timers with the user provided `key`.
Refers to openhab/openhab-core#2911.

Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
splatch pushed a commit to ConnectorIO/copybara-hab-core that referenced this pull request Jul 12, 2023
Signed-off-by: Jan N. Klug <github@klug.nrw>
GitOrigin-RevId: c2ec97c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement or new feature of the Core
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add identifying information to timers for errors Feature request: Set a custom title for rule timers
3 participants