Skip to content

provide serviceData on discover event#2

Closed
sethvincent wants to merge 11 commits into
mainfrom
service-data
Closed

provide serviceData on discover event#2
sethvincent wants to merge 11 commits into
mainfrom
service-data

Conversation

@sethvincent

@sethvincent sethvincent commented May 3, 2026

Copy link
Copy Markdown
Contributor

With this pr we can now read service data from a peripheral advertisement.

I added a distinct type for the object we get from the discover event.

I've made a matching pr on bare-bluetooth-apple.

Comment thread test/service-data.js Outdated
Comment thread binding.cc Outdated
@sethvincent sethvincent requested review from a team and kasperisager May 12, 2026 19:01
Comment thread lib/central.d.ts Outdated
Comment thread lib/peripheral.js Outdated
Comment thread lib/peripheral.js Outdated
Comment thread lib/central.js Outdated
sethvincent and others added 9 commits May 14, 2026 11:31
Signed-off-by: Tony Gorez <gorez.tony@gmail.com>
Signed-off-by: Tony Gorez <gorez.tony@gmail.com>
Signed-off-by: Tony Gorez <gorez.tony@gmail.com>
Signed-off-by: Tony Gorez <gorez.tony@gmail.com>
Signed-off-by: Tony Gorez <gorez.tony@gmail.com>
Signed-off-by: Tony Gorez <gorez.tony@gmail.com>
Signed-off-by: Tony Gorez <gorez.tony@gmail.com>
@sethvincent

Copy link
Copy Markdown
Contributor Author

The workflow now passes with the latest commit I added. It adds another static cast of JNIEnv that can probably be cleaned up in some way.

@sethvincent

Copy link
Copy Markdown
Contributor Author

closing this in favor of #5 and #6

@sethvincent sethvincent closed this Jun 8, 2026
@sethvincent sethvincent mentioned this pull request Jun 9, 2026
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.

3 participants