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

ScanSettings: Add setLegacy method #801

Merged
merged 1 commit into from
Jul 9, 2022

Conversation

danielstuart14
Copy link
Contributor

@danielstuart14 danielstuart14 commented Jul 5, 2022

This option allows for scanning of BLE devices with extended advertising.
It is supported on Android 8.0 and above.
https://developer.android.com/reference/android/bluetooth/le/ScanSettings.Builder#setLegacy(boolean)

Closes #774

This option allows for scanning of BLE devices with extended advertising.
It is supported on Android 8.0 and above.
https://developer.android.com/reference/android/bluetooth/le/ScanSettings.Builder#setLegacy(boolean)
@CLAassistant
Copy link

CLAassistant commented Jul 5, 2022

CLA assistant check
All committers have signed the CLA.

@danielstuart14
Copy link
Contributor Author

@dariuszseweryn, this solves #774 . Prior to this commit we couldn't read BLE 5 devices advertising with more than 31 bytes. I kept it disabled by default, as android does.

@danielstuart14
Copy link
Contributor Author

The image below shows an application reading an extended advertising (70 bytes on this case). I modified the flutter_reactive_ble library to use my fork of RxAndroidBLE.
image

@dariuszseweryn
Copy link
Owner

Looks good. I plan to merge it in the near future (read days)

@dariuszseweryn dariuszseweryn changed the base branch from master to develop July 9, 2022 19:18
@dariuszseweryn dariuszseweryn merged commit 4371b78 into dariuszseweryn:develop Jul 9, 2022
@dariuszseweryn
Copy link
Owner

I have merged it to develop branch — it will be available in a 1.16.0-SNAPSHOT shortly and a regular release a few days from now

@Littlebigdondon
Copy link

@dariuszseweryn Eagerly awaiting this update 🙏

@dariuszseweryn
Copy link
Owner

I have just released 1.16.0. Should be available in the nearest hours. Cheers!

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.

4 participants