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

new Version problem #127

Closed
arteck opened this issue Nov 9, 2018 · 11 comments
Closed

new Version problem #127

arteck opened this issue Nov 9, 2018 · 11 comments

Comments

@arteck
Copy link
Contributor

arteck commented Nov 9, 2018

after reinstalling the lib (to the new version 5.0.0) all my devices not work

@Koenkk
Copy link
Owner

Koenkk commented Nov 9, 2018

5.0.0 has a breaking change. Do you use zigbee2mqtt or iobroker?

@arteck
Copy link
Contributor Author

arteck commented Nov 12, 2018

iobroker

@Koenkk
Copy link
Owner

Koenkk commented Nov 12, 2018

@kirovilya can you make iobroker compatible with 5.0.0?

@kirovilya
Copy link
Contributor

@Koenkk :) yes, i can. but I see that it is not yet used in zigbee2mqtt - so I was not in a hurry :))

@Koenkk
Copy link
Owner

Koenkk commented Nov 12, 2018

It is Koenkk/zigbee2mqtt@9119e54

@kirovilya
Copy link
Contributor

Yes, I saw it. I will do support soon
@arteck wait :)

@arteck
Copy link
Contributor Author

arteck commented Nov 12, 2018

ok

@kirovilya
Copy link
Contributor

@arteck Updated adapter dev-version 0.8.0b
@Koenkk When I try 'get' command, I get error:
Error: Payload of commnad: read must have attrId property
I try change code from:

           } else if (type === 'get') {
                return {
                    cid: cid,
                    cmd: 'read',
                    cmdType: 'foundation',
                    zclData: [{attrId: zclId.attr(cid, attrId).value}],
                    cfg: cfg.default,
                };
            }

to

           } else if (type === 'get') {
                return {
                    cid: cid,
                    cmd: 'read',
                    cmdType: 'foundation',
                    zclData: {attrId: zclId.attr(cid, attrId).value},
                    cfg: cfg.default,
                };
            }

and it works

@Koenkk
Copy link
Owner

Koenkk commented Nov 13, 2018

@kirovilya did you apply this change: Koenkk/zigbee2mqtt@9119e54#diff-ac8e6bd5fdab1261687971dcc0eba984R217 (foundation without []).

@kirovilya
Copy link
Contributor

@Koenkk Oh, I missed it. Thank. I will correct

@arteck
Copy link
Contributor Author

arteck commented Nov 13, 2018

until now .. works

@arteck arteck closed this as completed Nov 14, 2018
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

3 participants