Add new BLE detectors#172
Draft
tototo31 wants to merge 12 commits intoGhostESP-Revival:Development-dekifrom
Draft
Add new BLE detectors#172tototo31 wants to merge 12 commits intoGhostESP-Revival:Development-dekifrom
tototo31 wants to merge 12 commits intoGhostESP-Revival:Development-dekifrom
Conversation
Member
|
since new detectors section has been made it makes sense to move skimmer there too |
Collaborator
Author
|
Totally move skimmer and probably find flippers but yea haven’t quite gotten that far with it yet 😅
I’ve only really done cli testing so far
Sent from [Proton Mail](https://proton.me/mail/home) for iOS.
…-------- Original Message --------
On Friday, 11/14/25 at 00:34 jay candel ***@***.***> wrote:
jaylikesbunda left a comment [(GhostESP-Revival/GhostESP#172)](#172 (comment))
since new detectors section has been made it makes sense to move skimmer there too
—
Reply to this email directly, [view it on GitHub](#172 (comment)), or [unsubscribe](https://github.com/notifications/unsubscribe-auth/AA4GNROB34UIOXYSNOU52QD34VZZBAVCNFSM6AAAAACL52L522VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTKMZRGEYTONJVGU).
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
…optimize memory usage on ESP32.
…zing "Detectors" for improved clarity and consistency.
Collaborator
Author
|
need to do some testing but if all works this should be ready to go. I just dont have any of these devices on hand so ill need to figure out the best way to test. Will probably need to spoof some MACs |
Member
|
I have successfully tested the Apple device detector and it works flawlessly. Immediately detected an iPhone as soon as i turned on BT; also picked up my airpods and my airtag i'm pretty sure |
Member
|
also might be worth noting here, there was an issue where we were registering handlers before ble init which would cause scanning to be very unreliable. i applied the fix here too under 'fix order of operations' |
Member
|
any reason this is stuck as a draft? |
Collaborator
Author
|
Cause a lot of it was untested
…-------- Original Message --------
On Saturday, 02/07/26 at 21:56 jay candel ***@***.***> wrote:
jaylikesbunda left a comment [(GhostESP-Revival/GhostESP#172)](#172 (comment))
any reason this is stuck as a draft?
—
Reply to this email directly, [view it on GitHub](#172 (comment)), or [unsubscribe](https://github.com/notifications/unsubscribe-auth/AA4GNRIO3KCG3LH7HOOCNY34K2XVHAVCNFSM6AAAAACL52L522VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTQNRWGA3TGMZQHA).
You are receiving this because you were assigned.Message ID: ***@***.***>
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What's new (Author - fill this out)
Changed
Removed
Verification (Author - fill this out)
#ifdef CONFIG_...or similarLinked Issues
Checklist (Reviewer - don't fill this out)