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

feat: add support for Linptech RS1BB(MI) #77

Merged
merged 9 commits into from
May 5, 2024

Conversation

kvakulo
Copy link
Contributor

@kvakulo kvakulo commented May 5, 2024

Attempt to add support for RS1BB(MI) in xiaomi-ble.

I'm not sure what is the best way to test this change, but I try to raise the PR anyway and will try to explore how to test it out.

This PR is based on the change made in ble_monitor by @Ernst79: custom-components/ble_monitor@2b5e8ea

tests/test_parser.py Outdated Show resolved Hide resolved
@Ernst79
Copy link
Collaborator

Ernst79 commented May 5, 2024

sorry, also without the xc0 at the end, that is the RSSI

tests/test_parser.py Outdated Show resolved Hide resolved
tests/test_parser.py Outdated Show resolved Hide resolved
@Ernst79
Copy link
Collaborator

Ernst79 commented May 5, 2024

Almost there, only some linting issues, just follow the suggested changes.

@kvakulo kvakulo changed the title Add support for Linptech RS1BB(MI) feat: add support for Linptech RS1BB(MI) May 5, 2024
@Ernst79
Copy link
Collaborator

Ernst79 commented May 5, 2024

Nice work, thanks.

@Ernst79 Ernst79 merged commit 57d070c into Bluetooth-Devices:main May 5, 2024
6 checks passed
@kvakulo
Copy link
Contributor Author

kvakulo commented May 5, 2024

Thanks for the live assistance with this PR, @Ernst79, I really appreciate it! ❤️

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

Successfully merging this pull request may close these issues.

2 participants