这是indexloc提供的服务,不要输入任何密码
Skip to content
This repository was archived by the owner on May 17, 2021. It is now read-only.

Conversation

@johgoe
Copy link
Contributor

@johgoe johgoe commented Jun 10, 2017

Tested OH1 maxcul binding with OH2 and added the feature description for this binding

Copy link
Member

@kaikreuzer kaikreuzer left a comment

Choose a reason for hiding this comment

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

Great, thanks!
Just two very small comments to bring it in line with the meta-data of other bindings.

xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xmlns:binding="http://eclipse.org/smarthome/schemas/binding/v1.0.0"
xsi:schemaLocation="http://eclipse.org/smarthome/schemas/binding/v1.0.0 http://eclipse.org/smarthome/schemas/binding-1.0.0.xsd">
<name>openHAB MaxCul Binding</name>
Copy link
Member

Choose a reason for hiding this comment

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

Please remove "openHAB" here

xsi:schemaLocation="http://eclipse.org/smarthome/schemas/binding/v1.0.0 http://eclipse.org/smarthome/schemas/binding-1.0.0.xsd">
<name>openHAB MaxCul Binding</name>
<description>The aim of this binding is to allow the connection from openHAB to MAX! devices (wall thermostat/radiator valves) using the [CUL USB dongle](http://busware.de/tiki-index.php?page=CUL) rather than the MAX!Cube.</description>
<author>Paul Hampson (cyclingengineer)</author>
Copy link
Member

Choose a reason for hiding this comment

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

remove "(cyclingengineer)"

@johgoe johgoe force-pushed the maxcul_openhab2_compatibility branch from 27405ef to aed651d Compare June 11, 2017 19:41
@johgoe
Copy link
Contributor Author

johgoe commented Jun 11, 2017

@kaikreuzer I changed the meta-data like you recommended.

Copy link
Member

@kaikreuzer kaikreuzer left a comment

Choose a reason for hiding this comment

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

Perfect, many thanks!

@kaikreuzer kaikreuzer merged commit 82f4e6d into openhab:master Jun 11, 2017
@9037568 9037568 added this to the 1.10.0 milestone Jun 25, 2017
@johgoe johgoe deleted the maxcul_openhab2_compatibility branch December 7, 2019 18:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants