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

Add bluetooth tracking #891

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

Lash-L
Copy link

@Lash-L Lash-L commented Jul 18, 2024

Making this as a draft as it is still a WIP, but I'm not sure of my immediate time to continue (might be next week before I finish this)

So I wanted to make this so you knew I was working on it and could give any early feedback if you wished.

Relates to #123

Based off of #590 but with some fundamental changes.

  1. Removed device class stuff as it felt like a lot of code to add for very little trade off.
  2. Instead of showing the closest device ( I could potentially add this back) - it shows the rssi of the devices you specify.

I developed this within HA Devcontainer, so some of HA styling was auto applied, I will remove it before this PR is completely done.

Will also have a PR for adding the ability to specify known hosts incase zeroconf doesn't work, as I had to work my way around that functionality for getting the devcontainer to work.

Still to do:

  • Add ability to track regular bluetooth mac address
  • Add ability to remove bluetooth mac address or irk mac address
  • Seperate Bluetooth polling, 120 is likely far too high for this and something like 5-10 would make more sense so a bluetooth only update interval would be sane imo

@Lash-L Lash-L marked this pull request as ready for review July 23, 2024 21:09
@Lash-L
Copy link
Author

Lash-L commented Jul 23, 2024

@leikoilja This should be ready for a review now :) Thanks!

@Lash-L
Copy link
Author

Lash-L commented Jul 25, 2024

Set it to create a Hascanner so that the information can be used by other integrations! I had to bump some versioning for it to work correctly

@Lash-L
Copy link
Author

Lash-L commented Jul 27, 2024

Removed habluetoothscanner as it was going to cause issues posting an advertisement without data

If in the future someone wants to look at that problem again, they can see my branch here: https://github.com/Lash-L/ha-google-home/tree/bluetooth_scanner_obj

@leikoilja
Copy link
Owner

@KapJI , @DurgNomis-drol , @Drakulix, @ArnyminerZ, any of you guys can help review and test this?
i currently have no google home device nearby to play around with.

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