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

GarageDoorOpenerAdvanced.js #138

Closed
c-o-m-m-a-n-d-e-r opened this issue Jan 21, 2019 · 7 comments
Closed

GarageDoorOpenerAdvanced.js #138

c-o-m-m-a-n-d-e-r opened this issue Jan 21, 2019 · 7 comments
Labels

Comments

@c-o-m-m-a-n-d-e-r
Copy link
Contributor

Hey,
in the GarageDoorOpenerAdvanced.js File in Line 464 ... i think it must be

if (field === "KNXDoorClose") {

or not ?

@giase82
Copy link
Contributor

giase82 commented Jan 22, 2019

Hi,

I don't think so... The field in the example below is:
{ "Type": "KNXDoorClosed", "Listen": "2/3/2", "DPT": "DPT1" },

Do I missunderstand your question?

//Christof

@giase82
Copy link
Contributor

giase82 commented Jan 22, 2019

However, I agree this is inconsistent with KNXDoorOpen. One could fix it the one or the other way around.

@c-o-m-m-a-n-d-e-r
Copy link
Contributor Author

I know this was in the Example ... but i was debugging the process an the Type "KNXDoorClosed" was never get called.

After changing the Type to "KNXDoorClose" without the "d" and changing Line 464 everything was working fine.

@giase82
Copy link
Contributor

giase82 commented May 22, 2020

Sorry for that. Fixed it. Struggling with the git-magic right now for creating branches, commit etc

@snowdd1
Copy link
Owner

snowdd1 commented May 23, 2020

Hi @giase82 , I think @c-o-m-m-a-n-d-e-r Paul already fixed that in PR #158
Right?

@giase82
Copy link
Contributor

giase82 commented May 23, 2020

Yes! Sorry I did forget ;)

@snowdd1 snowdd1 added the bug label May 23, 2020
@snowdd1
Copy link
Owner

snowdd1 commented May 23, 2020

closed with #158

@snowdd1 snowdd1 closed this as completed May 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants