-
-
Notifications
You must be signed in to change notification settings - Fork 243
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
Thing add page: Add support for optional scan input #2771
Conversation
Refs openhab/openhab-core#4389. Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
#2283 Bundle Size — 10.82MiB (+0.01%).1de9695(current) vs 217fc06 main#2280(baseline) Warning Bundle contains 2 duplicate packages – View duplicate packages Bundle metrics
Bundle size by type
Bundle analysis report Branch florian-h05:thing-add-scaninput Project dashboard Generated by RelativeCI Documentation Report issue |
Tested and it works exactly as we expected. Top. @digitaldan you can now rely on this new feature and remove pairing code as thing configuration and as thing channel. @florian-h05 I just saw a little bug in rendering with a text almost hidden: |
@lolodomo Thanks for noticing … this is cause by a „quick-and-dirty“ style fix I used, I should probably do it the clean way now. |
Follow-up for openhab#2771. Fixes openhab#2771 (comment). Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
Follow-up for #2771. Fixes #2771 (comment). Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
@florian-h05 : final step would be to also propose to scan a QR code. Is there something like that in "HTML" standard ? |
I have already played around with QR code scanners in the browser in a little side project … so it is possible. These pairing code QR codes only contain the pairing code itself or additional data? |
Refs openhab/openhab-core#4389.