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

[velbus] Fix typo in the Counter Channel name for VMB7IN #11553

Merged
merged 19 commits into from
Nov 12, 2021
Merged

[velbus] Fix typo in the Counter Channel name for VMB7IN #11553

merged 19 commits into from
Nov 12, 2021

Conversation

Rosen01
Copy link
Contributor

@Rosen01 Rosen01 commented Nov 10, 2021

Typo fixed.

Signed-off-by: Daniel Rosengarten [email protected]

Rosen01 and others added 19 commits May 9, 2021 22:29
New functionnality :
Add the the possibility to simulate the PRESSED and LONG PRESSED message of an input.

Module supported with button simulation :
VMB1RYS (button : CH6)
VMB6IN (buttons : CH1 ... CH6)
VMB2PBN, VMB6PBN, VMB7IN, VMB8IR, VMB8PB, VMB8PBU, VMBEL1, VMBEL2, VMBEL4, VMBGP1, VMBGP1-2, VMBGP2, VMBGP2-2, VMBGP4, VMBGP4-2, VMBGP4PIR, VMBGP4PIR-2 (buttons : CH1 ... CH8)
VMBELO, VMBGPOD, VMBGPOD-2 (buttons : CH1 ... CH32)

Fix bug :

The channels names were not correctly assigned to the thing properties. The last channel had the default name, not the one retrieved from the module.

Signed-off-by: Daniel Rosengarten <[email protected]>
Co-authored-by: cedricboon <[email protected]>
Signed-off-by: Daniel Rosengarten <[email protected]>
Co-authored-by: cedricboon <[email protected]>
Signed-off-by: Daniel Rosengarten <[email protected]>
Co-authored-by: cedricboon <[email protected]>
Signed-off-by: Daniel Rosengarten <[email protected]>
Remove uneeded  Thread.sleep in code.
Trigger the events PRESSED, LONG_PRESSED, RELEASED on the linked trigger channel when using the button simulation.

Signed-off-by: Daniel Rosengarten <[email protected]>
Redundant superinterface DiscoveryService for the type VelbusThingDiscoveryService, already defined by AbstractDiscoveryService.

Signed-off-by: Daniel Rosengarten <[email protected]>
Typo in the Counter Channel name for VMB7IN.

Signed-off-by: Daniel Rosengarten <[email protected]>
Copy link
Contributor

@lolodomo lolodomo left a comment

Choose a reason for hiding this comment

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

LGTM

@lolodomo lolodomo merged commit e98f7a9 into openhab:main Nov 12, 2021
@lolodomo lolodomo added the bug An unexpected problem or unintended behavior of an add-on label Nov 12, 2021
@lolodomo lolodomo added this to the 3.2 milestone Nov 12, 2021
@wborn wborn changed the title [velbus] Fix typo in the Counter Channel name for VMB7IN (#11521) [velbus] Fix typo in the Counter Channel name for VMB7IN Dec 18, 2021
NickWaterton pushed a commit to NickWaterton/openhab-addons that referenced this pull request Dec 30, 2021
…) (openhab#11553)

* [velbus] Add new functionality PRESSED and LONG PRESSED and fix bug

New functionnality :
Add the the possibility to simulate the PRESSED and LONG PRESSED message of an input.

Module supported with button simulation :
VMB1RYS (button : CH6)
VMB6IN (buttons : CH1 ... CH6)
VMB2PBN, VMB6PBN, VMB7IN, VMB8IR, VMB8PB, VMB8PBU, VMBEL1, VMBEL2, VMBEL4, VMBGP1, VMBGP1-2, VMBGP2, VMBGP2-2, VMBGP4, VMBGP4-2, VMBGP4PIR, VMBGP4PIR-2 (buttons : CH1 ... CH8)
VMBELO, VMBGPOD, VMBGPOD-2 (buttons : CH1 ... CH32)

Fix bug :

The channels names were not correctly assigned to the thing properties. The last channel had the default name, not the one retrieved from the module.

Signed-off-by: Daniel Rosengarten <[email protected]>

* Update bundles/org.openhab.binding.velbus/README.md

Co-authored-by: cedricboon <[email protected]>
Signed-off-by: Daniel Rosengarten <[email protected]>

* Update bundles/org.openhab.binding.velbus/README.md

Co-authored-by: cedricboon <[email protected]>
Signed-off-by: Daniel Rosengarten <[email protected]>

* Update bundles/org.openhab.binding.velbus/README.md

Co-authored-by: cedricboon <[email protected]>
Signed-off-by: Daniel Rosengarten <[email protected]>

* Add trigger on channel when using button simulation

Remove uneeded  Thread.sleep in code.
Trigger the events PRESSED, LONG_PRESSED, RELEASED on the linked trigger channel when using the button simulation.

Signed-off-by: Daniel Rosengarten <[email protected]>

* Fix redundant superinterface DiscoveryService

Redundant superinterface DiscoveryService for the type VelbusThingDiscoveryService, already defined by AbstractDiscoveryService.

Signed-off-by: Daniel Rosengarten <[email protected]>

* Fix bug openhab#11521

Typo in the Counter Channel name for VMB7IN.

Signed-off-by: Daniel Rosengarten <[email protected]>

Co-authored-by: cedricboon <[email protected]>
Signed-off-by: Nick Waterton <[email protected]>
mischmidt83 pushed a commit to mischmidt83/openhab-addons that referenced this pull request Jan 9, 2022
…) (openhab#11553)

* [velbus] Add new functionality PRESSED and LONG PRESSED and fix bug

New functionnality :
Add the the possibility to simulate the PRESSED and LONG PRESSED message of an input.

Module supported with button simulation :
VMB1RYS (button : CH6)
VMB6IN (buttons : CH1 ... CH6)
VMB2PBN, VMB6PBN, VMB7IN, VMB8IR, VMB8PB, VMB8PBU, VMBEL1, VMBEL2, VMBEL4, VMBGP1, VMBGP1-2, VMBGP2, VMBGP2-2, VMBGP4, VMBGP4-2, VMBGP4PIR, VMBGP4PIR-2 (buttons : CH1 ... CH8)
VMBELO, VMBGPOD, VMBGPOD-2 (buttons : CH1 ... CH32)

Fix bug :

The channels names were not correctly assigned to the thing properties. The last channel had the default name, not the one retrieved from the module.

Signed-off-by: Daniel Rosengarten <[email protected]>

* Update bundles/org.openhab.binding.velbus/README.md

Co-authored-by: cedricboon <[email protected]>
Signed-off-by: Daniel Rosengarten <[email protected]>

* Update bundles/org.openhab.binding.velbus/README.md

Co-authored-by: cedricboon <[email protected]>
Signed-off-by: Daniel Rosengarten <[email protected]>

* Update bundles/org.openhab.binding.velbus/README.md

Co-authored-by: cedricboon <[email protected]>
Signed-off-by: Daniel Rosengarten <[email protected]>

* Add trigger on channel when using button simulation

Remove uneeded  Thread.sleep in code.
Trigger the events PRESSED, LONG_PRESSED, RELEASED on the linked trigger channel when using the button simulation.

Signed-off-by: Daniel Rosengarten <[email protected]>

* Fix redundant superinterface DiscoveryService

Redundant superinterface DiscoveryService for the type VelbusThingDiscoveryService, already defined by AbstractDiscoveryService.

Signed-off-by: Daniel Rosengarten <[email protected]>

* Fix bug openhab#11521

Typo in the Counter Channel name for VMB7IN.

Signed-off-by: Daniel Rosengarten <[email protected]>

Co-authored-by: cedricboon <[email protected]>
Signed-off-by: Michael Schmidt <[email protected]>
nemerdaud pushed a commit to nemerdaud/openhab-addons that referenced this pull request Jan 28, 2022
…) (openhab#11553)

* [velbus] Add new functionality PRESSED and LONG PRESSED and fix bug

New functionnality :
Add the the possibility to simulate the PRESSED and LONG PRESSED message of an input.

Module supported with button simulation :
VMB1RYS (button : CH6)
VMB6IN (buttons : CH1 ... CH6)
VMB2PBN, VMB6PBN, VMB7IN, VMB8IR, VMB8PB, VMB8PBU, VMBEL1, VMBEL2, VMBEL4, VMBGP1, VMBGP1-2, VMBGP2, VMBGP2-2, VMBGP4, VMBGP4-2, VMBGP4PIR, VMBGP4PIR-2 (buttons : CH1 ... CH8)
VMBELO, VMBGPOD, VMBGPOD-2 (buttons : CH1 ... CH32)

Fix bug :

The channels names were not correctly assigned to the thing properties. The last channel had the default name, not the one retrieved from the module.

Signed-off-by: Daniel Rosengarten <[email protected]>

* Update bundles/org.openhab.binding.velbus/README.md

Co-authored-by: cedricboon <[email protected]>
Signed-off-by: Daniel Rosengarten <[email protected]>

* Update bundles/org.openhab.binding.velbus/README.md

Co-authored-by: cedricboon <[email protected]>
Signed-off-by: Daniel Rosengarten <[email protected]>

* Update bundles/org.openhab.binding.velbus/README.md

Co-authored-by: cedricboon <[email protected]>
Signed-off-by: Daniel Rosengarten <[email protected]>

* Add trigger on channel when using button simulation

Remove uneeded  Thread.sleep in code.
Trigger the events PRESSED, LONG_PRESSED, RELEASED on the linked trigger channel when using the button simulation.

Signed-off-by: Daniel Rosengarten <[email protected]>

* Fix redundant superinterface DiscoveryService

Redundant superinterface DiscoveryService for the type VelbusThingDiscoveryService, already defined by AbstractDiscoveryService.

Signed-off-by: Daniel Rosengarten <[email protected]>

* Fix bug openhab#11521

Typo in the Counter Channel name for VMB7IN.

Signed-off-by: Daniel Rosengarten <[email protected]>

Co-authored-by: cedricboon <[email protected]>
marcfischerboschio pushed a commit to bosch-io/openhab-addons that referenced this pull request May 5, 2022
…) (openhab#11553)

* [velbus] Add new functionality PRESSED and LONG PRESSED and fix bug

New functionnality :
Add the the possibility to simulate the PRESSED and LONG PRESSED message of an input.

Module supported with button simulation :
VMB1RYS (button : CH6)
VMB6IN (buttons : CH1 ... CH6)
VMB2PBN, VMB6PBN, VMB7IN, VMB8IR, VMB8PB, VMB8PBU, VMBEL1, VMBEL2, VMBEL4, VMBGP1, VMBGP1-2, VMBGP2, VMBGP2-2, VMBGP4, VMBGP4-2, VMBGP4PIR, VMBGP4PIR-2 (buttons : CH1 ... CH8)
VMBELO, VMBGPOD, VMBGPOD-2 (buttons : CH1 ... CH32)

Fix bug :

The channels names were not correctly assigned to the thing properties. The last channel had the default name, not the one retrieved from the module.

Signed-off-by: Daniel Rosengarten <[email protected]>

* Update bundles/org.openhab.binding.velbus/README.md

Co-authored-by: cedricboon <[email protected]>
Signed-off-by: Daniel Rosengarten <[email protected]>

* Update bundles/org.openhab.binding.velbus/README.md

Co-authored-by: cedricboon <[email protected]>
Signed-off-by: Daniel Rosengarten <[email protected]>

* Update bundles/org.openhab.binding.velbus/README.md

Co-authored-by: cedricboon <[email protected]>
Signed-off-by: Daniel Rosengarten <[email protected]>

* Add trigger on channel when using button simulation

Remove uneeded  Thread.sleep in code.
Trigger the events PRESSED, LONG_PRESSED, RELEASED on the linked trigger channel when using the button simulation.

Signed-off-by: Daniel Rosengarten <[email protected]>

* Fix redundant superinterface DiscoveryService

Redundant superinterface DiscoveryService for the type VelbusThingDiscoveryService, already defined by AbstractDiscoveryService.

Signed-off-by: Daniel Rosengarten <[email protected]>

* Fix bug openhab#11521

Typo in the Counter Channel name for VMB7IN.

Signed-off-by: Daniel Rosengarten <[email protected]>

Co-authored-by: cedricboon <[email protected]>
andan67 pushed a commit to andan67/openhab-addons that referenced this pull request Nov 6, 2022
…) (openhab#11553)

* [velbus] Add new functionality PRESSED and LONG PRESSED and fix bug

New functionnality :
Add the the possibility to simulate the PRESSED and LONG PRESSED message of an input.

Module supported with button simulation :
VMB1RYS (button : CH6)
VMB6IN (buttons : CH1 ... CH6)
VMB2PBN, VMB6PBN, VMB7IN, VMB8IR, VMB8PB, VMB8PBU, VMBEL1, VMBEL2, VMBEL4, VMBGP1, VMBGP1-2, VMBGP2, VMBGP2-2, VMBGP4, VMBGP4-2, VMBGP4PIR, VMBGP4PIR-2 (buttons : CH1 ... CH8)
VMBELO, VMBGPOD, VMBGPOD-2 (buttons : CH1 ... CH32)

Fix bug :

The channels names were not correctly assigned to the thing properties. The last channel had the default name, not the one retrieved from the module.

Signed-off-by: Daniel Rosengarten <[email protected]>

* Update bundles/org.openhab.binding.velbus/README.md

Co-authored-by: cedricboon <[email protected]>
Signed-off-by: Daniel Rosengarten <[email protected]>

* Update bundles/org.openhab.binding.velbus/README.md

Co-authored-by: cedricboon <[email protected]>
Signed-off-by: Daniel Rosengarten <[email protected]>

* Update bundles/org.openhab.binding.velbus/README.md

Co-authored-by: cedricboon <[email protected]>
Signed-off-by: Daniel Rosengarten <[email protected]>

* Add trigger on channel when using button simulation

Remove uneeded  Thread.sleep in code.
Trigger the events PRESSED, LONG_PRESSED, RELEASED on the linked trigger channel when using the button simulation.

Signed-off-by: Daniel Rosengarten <[email protected]>

* Fix redundant superinterface DiscoveryService

Redundant superinterface DiscoveryService for the type VelbusThingDiscoveryService, already defined by AbstractDiscoveryService.

Signed-off-by: Daniel Rosengarten <[email protected]>

* Fix bug openhab#11521

Typo in the Counter Channel name for VMB7IN.

Signed-off-by: Daniel Rosengarten <[email protected]>

Co-authored-by: cedricboon <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug An unexpected problem or unintended behavior of an add-on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants