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

add config.schema.json to get a UI configuration dialog #36

Merged
merged 6 commits into from
Jan 5, 2020

Conversation

lisanet
Copy link

@lisanet lisanet commented Dec 30, 2019

This add a configuration dialog for the homebridge-config-ui-x plugin, so that users can easily configure the eveatmo plugin via the config web UI.

@skrollme
Copy link
Owner

Hello @lisanet , thank you for your contribution. I will try to check this out next weekend and merge the PR.

@skrollme skrollme added this to the 0.5.0 milestone Dec 31, 2019
@skrollme skrollme self-assigned this Dec 31, 2019
…er manual, that a calibration will use this lower level. It seems that Eve uses a different value in its sensors, see prior commit.
@skrollme
Copy link
Owner

skrollme commented Jan 4, 2020

Hello @lisanet , I'm currently testing your PR and everything works fine except one startup-error here:

https://github.com/lisanet/homebridge-eveatmo/blob/master/device/weatherstation-device.js#L60

[2020-1-4 15:14:28] ReferenceError: NULL is not defined
    at WeatherstationDeviceType.<anonymous> (/usr/lib/node_modules/homebridge-eveatmo/device/weatherstation-device.js:60:14)

If I change the NULL to null, everything works. Maybe a different node-version (I'm using 8.4). If you want I can change this and merge your PR.

@skrollme
Copy link
Owner

skrollme commented Jan 4, 2020

... one more thing: It would also be great if you can change the PR target branch to the develop-branch.

@lisanet lisanet changed the base branch from master to develop January 5, 2020 10:56
@lisanet
Copy link
Author

lisanet commented Jan 5, 2020

ok, I've changed the PR to target your develop branch.

@skrollme skrollme merged commit 34cdcf7 into skrollme:develop Jan 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants