-
Notifications
You must be signed in to change notification settings - Fork 287
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
Adding TeslaFi Adapter to latest repository #4236
base: master
Are you sure you want to change the base?
Conversation
jsonConfig looks good... |
"xs" should specify a value of "12" at admin/jsonConfig.json/items/_StandardTab/_donate -> has to be 6, otherwise button is too big I'm no friend of automated translations - so I don't use i18n |
Automated adapter checkerioBroker.teslafi👍 No errors found
Add comment "RE-CHECK!" to start check anew |
Your TOO fast :-) I18n is a warning only which will not be considered blocking. |
:) - so all this 5 times 12 for dividers could be removed?.... -> next release ;) ... so must be fixed label isn't relevant anymore? Or do I have to change something? |
ioBroker repository information about New at LATEST taggingThanks 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. And please check the following information if not yet done:
You will find the results of the review and eventually issues / suggestions as a comment to this PR. So please keep this PR watched. If you have any urgent questions feel free to ask. @simatec Please take a look in respect to responsive design. Thanks |
Sorry, but due to travel / absence I will not able to do the review before ioBroker Meeting Solingen next weekend. |
No description provided.