Skip to content

Commit

Permalink
Merge pull request #41 from Istaroth/coolcover
Browse files Browse the repository at this point in the history
Remove unnecessary stop before up/down
  • Loading branch information
ttherbrink authored Dec 3, 2024
2 parents 218cb6b + abdd654 commit 95b11c6
Showing 1 changed file with 0 additions and 7 deletions.
7 changes: 0 additions & 7 deletions drivers/knx_windowcoverings/device.js
Original file line number Diff line number Diff line change
Expand Up @@ -46,10 +46,6 @@ class KNXWindowCovering extends KNXGenericDevice {
if (this.knxInterface) {
switch (value) {
case 'up':
if (this.settings.ga_stop) {
this.knxInterface.writeKNXGroupAddress(this.settings.ga_stop, 0, 'DPT1');
}

if (this.settings.invert_updown === true) {
return this.knxInterface.writeKNXGroupAddress(this.settings.ga_up_down, 1, 'DPT1')
.catch((knxerror) => {
Expand All @@ -61,9 +57,6 @@ class KNXWindowCovering extends KNXGenericDevice {
throw new Error(this.homey.__('errors.windowcovering_failed'), knxerror);
});
case 'down':
if (this.settings.ga_stop) {
this.knxInterface.writeKNXGroupAddress(this.settings.ga_stop, 0, 'DPT1');
}
if (this.settings.invert_updown === true) {
return this.knxInterface.writeKNXGroupAddress(this.settings.ga_up_down, 0, 'DPT1')
.catch((knxerror) => {
Expand Down

0 comments on commit 95b11c6

Please sign in to comment.