Skip to content
This repository has been archived by the owner on May 17, 2021. It is now read-only.

adapted rrd4j chart servlet to be compatible with openHAB 2 #2300

Merged
merged 1 commit into from
Mar 15, 2015

Conversation

kaikreuzer
Copy link
Member

Signed-off-by: Kai Kreuzer [email protected] (github: @kaikreuzer)

@kaikreuzer
Copy link
Member Author

@theoweiss This PR reverts your change of smarthome.userdata to openhab.userdata, because that broke the openHAB 2 compatibility (and such the feature for which this line of code had been added at all). I am not sure, whether you need this for the deb packages in some way, but if so, you would have to come up with some other solution.

@buildhive
Copy link

openhab » openhab #2540 SUCCESS
This pull request looks good
(what's this?)

@theoweiss
Copy link
Member

@kaikreuzer I changed this on request of @teichsta #2141 (comment)
No problem to revert this change. But we should also adapt db4o, may be the new mapdb persistence plugin and the startup scripts. I can care for the startup scripts and db4o if you want me to.

@kaikreuzer
Copy link
Member Author

But we should also adapt db4o, may be the new mapdb persistence plugin and the startup scripts. I can care for the startup scripts and db4o if you want me to.

Yes, this would be great if you could test them on openHAB 2 and do the necessary changes! Don't forget to update the compat matrix afterwards.

@theoweiss
Copy link
Member

OK. Face openHAB 2 IDE setup ;-)

@teichsta
Copy link
Member

No problem to revert this change.

thanks @theoweiss, sorry for the inconvenience!

@teichsta teichsta added this to the 1.7.0 milestone Mar 15, 2015
teichsta added a commit that referenced this pull request Mar 15, 2015
adapted rrd4j chart servlet to be compatible with openHAB 2
@teichsta teichsta merged commit a153e55 into openhab:master Mar 15, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants