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

BSIM test - RPA expired test- To add adv sets which will return false for the first set and true for the second on rpa_expired cb #69969

Closed
niym-ot opened this issue Mar 8, 2024 · 3 comments · Fixed by #70364
Assignees
Labels
area: Bluetooth Enhancement Changes/Updates/Additions to existing features

Comments

@niym-ot
Copy link
Contributor

niym-ot commented Mar 8, 2024

Could you post a follow-up PR where you add another id w/ two other adv sets, that return false for the first one and true for the second one? Ie started in reverse order so the iterator hits the false first.

Originally posted by @jori-nordic in #68892 (comment)

Copy link

github-actions bot commented Mar 8, 2024

Hi @niym-ot! We appreciate you submitting your first issue for our open-source project. 🌟

Even though I'm a bot, I can assure you that the whole community is genuinely grateful for your time and effort. 🤖💙

@jukkar
Copy link
Member

jukkar commented Mar 8, 2024

@niym-ot could you fix the PR title something shorter, now it contains basically the description of this issue.

@niym-ot niym-ot changed the title Add another id w/ two other adv sets, that return false for the first one and true for the second one? Ie started in reverse order so the iterator hits the false first. BSIM test - RPA expired test- To add adv sets which will return false for the first set and true for the second Mar 8, 2024
@niym-ot niym-ot changed the title BSIM test - RPA expired test- To add adv sets which will return false for the first set and true for the second BSIM test - RPA expired test- To add adv sets which will return false for the first set and true for the second on rpa_expired cb Mar 8, 2024
@henrikbrixandersen
Copy link
Member

Please use our bug template when reporting bugs. You need to edit this issue to include the information requested in https://github.com/zephyrproject-rtos/zephyr/blob/main/.github/ISSUE_TEMPLATE/001_bug_report.md

@nashif nashif added bug The issue is a bug, or the PR is fixing a bug priority: low Low impact/importance bug area: Bluetooth labels Mar 12, 2024
@aescolar aescolar added Enhancement Changes/Updates/Additions to existing features and removed bug The issue is a bug, or the PR is fixing a bug priority: low Low impact/importance bug labels Mar 12, 2024
@aescolar aescolar assigned niym-ot and unassigned jori-nordic Mar 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Bluetooth Enhancement Changes/Updates/Additions to existing features
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants