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

[senseBox] Forgot to add binding to feature.xml #2371

Merged

Conversation

hakan42
Copy link
Contributor

@hakan42 hakan42 commented Jun 14, 2017

Somehow, the commit adding this binding to feature.xml must have got lost before starting the pull requests 😊

@kaikreuzer
Copy link
Member

Weird... Didn't you miss it in the distro then? Thanks for adding it now!

@kaikreuzer kaikreuzer merged commit 1b22a92 into openhab:master Jun 14, 2017
@martinvw
Copy link
Member

Hmm, guys, I think it's worse 👎

It's not in the pom.xml either.

@martinvw
Copy link
Member

My excuse is that I was on the IoT tech days while reviewing ;-)

@hakan42
Copy link
Contributor Author

hakan42 commented Jun 14, 2017

Weird... Didn't you miss it in the distro then? Thanks for adding it now!

On my system, I was always using the self-compiled one... Actually, it's the first time I am checking the full distribution 😊

It's not in the pom.xml either.

Really? I'll check immediately and create a new PR... More things to check before I start a PR, I guess...

@hakan42
Copy link
Contributor Author

hakan42 commented Jun 14, 2017

Here it is: https://github.com/openhab/openhab2-addons/pull/2372

And this time, I ran a full build on my machine to make sure the binding is being picked up for compilation 😊

holmes added a commit to holmes/openhab2-addons that referenced this pull request Jun 16, 2017
* master:
  [senseBox] Forgot to add binding to feature.xml (openhab#2371)
  [Kodi] Fix audio sink (openhab#2301) (openhab#2303)
  Initial contribution of a Niko Home Control binding. (openhab#1589)
  [keba] fixed typo (openhab#2370)
  Removed utilisation of org.apache.http.client (openhab#2369)
  [RFXCOM] Make LWRF mood buttons work (openhab#2366)
  [senseBox] Switch caching to ESH ExpiringCacheMap (openhab#2361)
  Fixed problems found by AuthorTagCheck in the cometvisu bundle (openhab#2364)
  Added author tag where missing. (openhab#2351)
  [RFXCOM] Fix chill temperature (openhab#2362)
  initial contribution Windcentrale binding (openhab#1535)
  Tesla : reset the Even Stream connection to the Tesla Back-end if the event time stamps differ too much from the current system time (openhab#2358)
  Homematic: Some updates (openhab#2346)
  [RFXCOM] Support all data from wind sensors (openhab#2329)
  Gardena: Optimized refresh after command (openhab#2353)
  Fix some javadocs. (openhab#2349)
  Some small non-standard copyright banners which we missed while reviewing (openhab#2347)
@kaikreuzer kaikreuzer modified the milestone: 2.1 Jun 23, 2017
falkena pushed a commit to falkena/openhab-addons that referenced this pull request Jun 29, 2017
qvistgaard pushed a commit to qvistgaard/openhab2-addons that referenced this pull request Jun 30, 2017
jsjames pushed a commit to jsjames/openhab-addons that referenced this pull request Jul 1, 2017
reyem pushed a commit to reyem/openhab2-addons that referenced this pull request Jul 1, 2017
Markinus pushed a commit to Markinus/openhab2-addons that referenced this pull request Jul 2, 2017
ppieczul pushed a commit to ppieczul/openhab2-addons that referenced this pull request Jul 2, 2017
aogorek pushed a commit to aogorek/openhab2-addons that referenced this pull request Jul 5, 2017
Markinus pushed a commit to Markinus/openhab2-addons that referenced this pull request Sep 8, 2017
hillmanr pushed a commit to hillmanr/openhab2-addons-pollytts that referenced this pull request Dec 6, 2017
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.

3 participants