Skip to content

Comments

[tibber] Trigger event when day-ahead prices are available #19598

Merged
jlaur merged 1 commit intoopenhab:mainfrom
weymann:feature/tibber-price-event
Nov 3, 2025
Merged

[tibber] Trigger event when day-ahead prices are available #19598
jlaur merged 1 commit intoopenhab:mainfrom
weymann:feature/tibber-price-event

Conversation

@weymann
Copy link
Member

@weymann weymann commented Oct 31, 2025

Feature request from community:
If day-ahead prices are fetched successfully an event is triggered via new trigger channel event in price group.

@weymann weymann requested a review from kjoglum as a code owner October 31, 2025 15:29
@weymann weymann requested a review from Copilot October 31, 2025 15:30
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a new trigger channel to the Tibber binding that fires an event when day-ahead prices are successfully fetched. The change introduces version 7 of the thing type with the new event trigger channel.

  • Adds a new trigger channel event that fires DAY_AHEAD_PRICE_UPDATE events
  • Updates thing type version from 6 to 7 with corresponding migration instructions
  • Implements trigger logic in TibberHandler to fire event when tomorrow's prices are newly available

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
instructions.xml Adds migration instructions for version 7 to add the new event channel
thing-types.xml Increments thingTypeVersion property from 6 to 7
price-group.xml Adds the event channel to the price channel group
channel-types.xml Defines the new price-event trigger channel type with DAY_AHEAD_PRICE_UPDATE event
TibberHandler.java Implements logic to trigger the event when new day-ahead prices are fetched
TibberBindingConstants.java Adds constants for the new event channel and event value
README.md Documents the new event channel and provides usage example

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@jlaur jlaur added the enhancement An enhancement or new feature for an existing add-on label Oct 31, 2025
Copy link
Contributor

@jlaur jlaur left a comment

Choose a reason for hiding this comment

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

Thanks! Only two minor comments, otherwise LGTM.

Signed-off-by: Bernd Weymann <bernd.weymann@gmail.com>

fix copilot finding

Signed-off-by: Bernd Weymann <bernd.weymann@gmail.com>

add price update event

Signed-off-by: Bernd Weymann <bernd.weymann@gmail.com>
@weymann weymann force-pushed the feature/tibber-price-event branch from 7123547 to cce7345 Compare November 2, 2025 15:12
Copy link
Contributor

@jlaur jlaur left a comment

Choose a reason for hiding this comment

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

LGTM

@jlaur jlaur merged commit f0dcc72 into openhab:main Nov 3, 2025
2 checks passed
@jlaur jlaur added this to the 5.1 milestone Nov 3, 2025
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 for an existing add-on

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants