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

[compat1x] Fix ChartProvider API break #215

Merged

Conversation

hreichert
Copy link
Contributor

The ESH PR eclipse-archived/smarthome#4291 introduced a change in the ChartProvider API.
This commit changes the compat1x ChartProviderDelegate to compile with these changes.

This should be merged when a new ESH stable is built, otherwise it would cause build failure of openhab-core.

The ESH PR 4291 introduced a change in the ChartProvider API.
This commit changes the compat1x ChartProviderDelegate to work with these changes.

Signed-off-by: Holger Reichert <mail@h0lger.de>
@hreichert
Copy link
Contributor Author

hreichert commented Sep 25, 2017

Travis build is failing because it builds against the current ESH stable version, that does not contain the changes from eclipse-archived/smarthome#4291 yet.

@kaikreuzer
Copy link
Member

ESH Stable has been built, Travis is green 👍

@kaikreuzer kaikreuzer merged commit d5841d3 into openhab:master Sep 26, 2017
@hreichert hreichert deleted the compat1x-chartprovider-api-break branch September 26, 2017 11:08
@kaikreuzer kaikreuzer added this to the 2.2 milestone Dec 15, 2017
Rosi2143 pushed a commit to Rosi2143/openhab-core that referenced this pull request Dec 26, 2020
Signed-off-by: Thomas Dietrich <thomas.dietrich@tu-ilmenau.de> (github: ThomDietrich)
splatch pushed a commit to ConnectorIO/copybara-hab-core that referenced this pull request Jul 11, 2023
The ESH PR 4291 introduced a change in the ChartProvider API.
This commit changes the compat1x ChartProviderDelegate to work with these changes.

Signed-off-by: Holger Reichert <mail@h0lger.de>
GitOrigin-RevId: d5841d3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants