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

GamaBit Ltd. Window/Door Sensor #4189

Closed
Kristian8606 opened this issue Jan 13, 2021 · 24 comments · Fixed by #4215
Closed

GamaBit Ltd. Window/Door Sensor #4189

Kristian8606 opened this issue Jan 13, 2021 · 24 comments · Fixed by #4215

Comments

@Kristian8606
Copy link

Kristian8606 commented Jan 13, 2021

Device

  • Product name: Contact Switch.
  • Manufacturer: GamaBit Ltd.
  • Model identifier: GMB-HAS-DW-B01.
  • Device type : Please remove all unrelated device types.
  • Sensor

Screenshots

Screenshot 2021-01-14 at 0 49 59
Screenshot 2021-01-14 at 0 50 19
Screenshot 2021-01-14 at 0 51 45

@Kristian8606
Copy link
Author

Do i need to clone your repository to test the sensor?

@SwoopX
Copy link
Collaborator

SwoopX commented Jan 18, 2021

That would be indeed good if you could test it upfront.

@Kristian8606
Copy link
Author

Not yet visible in Phoscon

@SwoopX
Copy link
Collaborator

SwoopX commented Jan 18, 2021

That's because deconz is not Phoscon. It's the REST API what matters here.

For visibility in Phoscon, you'd need to request it in their repository.

@Kristian8606
Copy link
Author

I tried GET API request ip: 80/api/67CE474F7D/sensors but I don't see it, i see all the other sensors i have.

@SwoopX
Copy link
Collaborator

SwoopX commented Jan 18, 2021

Have you re-paired it?

@Kristian8606
Copy link
Author

Yes I tried twice, the different I see is that there is now a line in the deconz GUI connected to two bulbs as well as a connection to the conbee

@SwoopX
Copy link
Collaborator

SwoopX commented Jan 18, 2021

How does the node look like in deconz GUI?

@Kristian8606
Copy link
Author

I tried a third time. I do the search through Phoscon.
Screenshot 2021-01-18 at 13 38 44

@Kristian8606
Copy link
Author

How could I find it in a database to make sure it's there?

@Kristian8606
Copy link
Author

GMB-HAS-WD-B01 I noticed a discrepancy in the Model identifier
Screenshot 2021-01-18 at 13 46 55

@Kristian8606
Copy link
Author

If this is a mistake I think I wrote it wrong GMB-HAS-WD-B01 instead GMB-HAS-DW-B01

@Kristian8606
Copy link
Author

I fixed it sorry.

@SwoopX
Copy link
Collaborator

SwoopX commented Jan 18, 2021

Ah, a classic ;)

So, does it work now?

@Kristian8606
Copy link
Author

I fixed it in the title of the post but I expect you to fix it in the code.

@SwoopX
Copy link
Collaborator

SwoopX commented Jan 18, 2021

You wanted to say that you'd appreciate if I could change it in the code, I assume.

Yes, I already did that.

@Kristian8606
Copy link
Author

Yes, is are great you :)

@Kristian8606
Copy link
Author

Looks like I'll have to wait for the next beta. I'm compiling your storage but I can't see this sensor via an API request.

@SwoopX
Copy link
Collaborator

SwoopX commented Jan 18, 2021

So, you're saying it's not working, even with the amendments?

Just to be sure, you cloned the right branch?

@Kristian8606
Copy link
Author

I follow the instructions in the description, the only difference is that I copy and clone your storage https://github.com/SwoopX/deconz-rest-plugin.git is this the right way?

@SwoopX
Copy link
Collaborator

SwoopX commented Jan 18, 2021

No, it isn't. You must specifically clone and compile the branch I made the changes on:

git clone --branch=gamabit_oc https://github.com/SwoopX/deconz-rest-plugin.git gamabit
cd gamabit

@Kristian8606
Copy link
Author

The sensor is not yet exposed to the API.
I will try a second time to be sure and I will report.

AF851A6F-7842-424E-8B5E-5E704F0CE998

@Mimiix
Copy link
Collaborator

Mimiix commented Feb 24, 2021

The sensor is not yet exposed to the API.
I will try a second time to be sure and I will report.

AF851A6F-7842-424E-8B5E-5E704F0CE998

It seems not to be paired correctly.

@Kristian8606
Copy link
Author

Sorry the mistake is mine.
I added it again and now it works.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants