-
-
Notifications
You must be signed in to change notification settings - Fork 637
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
Detect Humanware Brailliant BI 40X and 20X via both USB and Bluetooth #11819
Conversation
3afa9b6
to
09bbd29
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do these devices also have serial support like the previous generation?
@LeonarddeR I don't know. I was not informed of that. |
It might be worth asking for this explicitly, though I don't insist.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks fine to me. If we could get explicit confirmation from someone that this works before the release, that would be great.
561a375
See test results for failed build of commit e212c4a0f3 |
I can see the error in the system test, silly off by one error. New PR coming soon. |
@michaelDCurran Diid you intent to merge this? |
I think this should go in 2020.4 if possible. |
@LeonarddeR Mint adding this to the 2020.4 milestone to avoid this being forgotten before the release? |
There is some confusion remaining around whether these devices are released yet. I'm waiting on clarification from Humanware on what to do. |
Is there any danger to including this in 2020.4 even if the devices aren't released yet? |
The issue is more a marketing issue about it being announced widely in
our documentation before Humanware did.
|
hi @michaelDCurran |
561a375
to
2733c7c
Compare
@feerrenrut I have retargetted this branch for beta now. Needs a review approval from you if you are happy. Humanware has confirmed this is oaky to go into NVDA 2020.4. Don't think they will provide us with any timely testing, nor we or anyone we know have access to one of those devices as they have not been released yet. However, the change is just adding extra mappings to more USB / Bluetooth IDs that use the same Humanware protocol. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Thanks @michaelDCurran.
Link to issue number:
None.
Summary of the issue:
Humanware BI 40X and 20X devices should be automatically detected via USB and Bluetooth.
Description of how this pull request fixes the issue:
Maps their respective USB IDs and Bluetooth names to our existing brailliantB driver.
Testing performed:
None - we don't have access to the devices, but we are informed these are compatible with the other Brailliant BI devices.
Known issues with pull request:
None.
Change log entry:
New features: