-
Notifications
You must be signed in to change notification settings - Fork 178
Extended advertising added to Gap docs #833
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
Extended advertising added to Gap docs #833
Conversation
@paul-szczepanek-arm Please add a brief description, and link to the code PR upon which this is dependent. |
Make minor copy edits.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work on this 👍
This PR is waiting on engineering review and a code dependency. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might be worth adding a warning about advertising extensions support being optional in Bluetooth 5, otherwise LGTM!
@donatieng I addressed your comment, please approve |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @paul-szczepanek-arm !
Thank you! Looking forward to having this merged to use it during OOB :) |
Update the docs to reflect added BT5 functionality (extended and periodic advertising) introduced in:
ARMmbed/mbed-os#8738