-
Notifications
You must be signed in to change notification settings - Fork 284
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 google-home-nest to latest #3615
base: master
Are you sure you want to change the base?
Conversation
Thanks for spending your time and providing a new adapter for ioBroker. Your adapter will get a manual review as soon as possible. Please stand by - this might last one or two weeks. Feel free to continue your work and create new releases. You do NOT need to close or update this PR in case of new releases. In the meantime please check any feedback issues logged by automatic adapter checker and try to fix them. You will find the results of the review and eventually issues / suggestings as a comment to this PR. So please keep this PR watched. If you have any urgent questions feel free to ask. reminder 22.05.2024 |
Automated adapter checkerioBroker.google-home-nest👍 No errors found
Add comment "RE-CHECK!" to start check anew |
First of all - THANK YOU for the time and effort you spend to maintain this adapter. I would like to give some feedback based on my personal oppinion. @Apollon77 might have additional suggestions or even a different oppinion to one or the other statement. Please feel free to contact him if you cannot follow my suggestions or want to discuss some special aspects.
Looks like some parts of the object / state id are retrieved from external sources, i.e. the 'deviceId' Those parts (or the complete id) must be filtered to not contain any forbidden character.
Thanks for reading and evaluating this suggestions. Please add a comment when you have reviewed and fixed the suggestionsor at least commented the suggestions and you think the adapter is ready for a re-review! reminder 5.6.2024 |
Thank you for the review and for the reply. As I mentioned in the adapter request repo., I am not a professional programmer. Therefore I knew that my code would probably be far away from perfect. On top of that, I sadly have zero javascript experience (besides the code I wrote for this adapter). So thanks again for putting me in the right direction. You probably scratched your head a lot while reviewing my code and I want to apologize for the inconvenience. I will try to fix & improve all mentioned aspects of my code in the following days. Thanks again. |
No problem. Most of devs here do this at their free time and for fun.
You code look very well. There are only seveal iobroker specific adaptions to be done. You will make this for sure. Please feel free to ask whenever you have a question. Please feel free to subscribe out telegram developer channels. Invites available at http://www.iobroker.dev. |
Any news? Please feel free to ask whenever you have a question. Please feel free to subscribe out telegram developer channels. Invites available at http://www.iobroker.dev/. Please add a comment when you have reviewed and fixed the suggestionsor at least commented the suggestions and you think the adapter is ready for a re-review! reminder 21.6.2024 |
Hey. |
reminder 13.7.2024 |
No description provided.