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

Working on V2(Beta) of homebridge? #132

Open
banboobee opened this issue Sep 25, 2024 · 4 comments
Open

Working on V2(Beta) of homebridge? #132

banboobee opened this issue Sep 25, 2024 · 4 comments

Comments

@banboobee
Copy link
Contributor

Hi developers,

I found this library doesn't work on V2 of homebridge.

Keys of hap.Perms object are renamed to PAIRED_READ/PAIRED_WRITE from READ/WRITE. These changes cause Homekit to reject the response from "GET /acceraries", since corresponding values are null, and all existing accessories become "not responding".

Unfortunately, I don't have any ideas to deal with this issue other than ad-hock approach. I edited fakegato-history.js to refer the keys like this,

Perms.PAIRED_READ || Perms.READ
Perms.PAIRED_WRITE || Perms.WRITE

With these changes, beta version of V2 homebridge begins to work.

So, I would like to ask,

Are there anyone facing similar issue?
Is this temporally issue on V2 developing, or libraries need to handle?
If handling is essential, are there better ways to deal?

@NorthernMan54
Copy link
Collaborator

This is noted, going to be reviewing this and other components for HB 2.0 readiness this month

@banboobee
Copy link
Contributor Author

Thank you for your reply.

Yes. I hope potential show stoppers will be idetified strategically.

@bwp91
Copy link

bwp91 commented Oct 1, 2024

Following the commit history of this repo, it looks like the fixes have already been made in v0.6.5.

https://github.com/simont77/fakegato-history/commits/master/

I'm just unsure if a newer version of the package has been published to npm with the very latest fix commit.

@simont77
Copy link
Owner

simont77 commented Oct 1, 2024

Yes, it is on npm https://www.npmjs.com/package/fakegato-history

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

No branches or pull requests

4 participants