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

bluetooth: fast_pair: separate subsequent pairing feature into Kconfig #16287

Merged
merged 3 commits into from
Jul 15, 2024

Conversation

kapi-no
Copy link
Contributor

@kapi-no kapi-no commented Jul 1, 2024

Added a new Kconfig option for the Fast Pair Subsequent Pairing feature. As a result, this previously supported feature is now configurable and can be disabled for the locator tag use case, which does not support subsequent pairing.

Aligned the Fast Pair Advertising Provider with the new Kconfig option for the Fast Pair service, which controls the Subsequent Pairing feature.

Removed the unavailable API call of the Fast Pair Advertising Provider from the Fast Pair Locator Tag sample.

Ref: NCSDK-27955

@github-actions github-actions bot added doc-required PR must not be merged without tech writer approval. changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Jul 1, 2024
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Jul 1, 2024

Test specification

CI/Jenkins/NRF

  • Integration Platforms

CI/Jenkins/integration

Test Module File based changes Manually selected West overwrite
desktop52_verification X
test-fw-nrfconnect-ble_samples X
test-sdk-find-my X
test_ble_commit X

Detailed information of selected test modules

Note: This message is automatically posted and updated by the CI

@kapi-no kapi-no force-pushed the fast_pair_subsequent_pairing branch from 87621cb to eab96fb Compare July 1, 2024 14:09
@NordicBuilder
Copy link
Contributor

You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds.

Note: This comment is automatically posted by the Documentation Publishing GitHub Action.

doc/nrf/external_comp/bt_fast_pair.rst Outdated Show resolved Hide resolved
help
Enable support for the Subsequent Pairing feature. Subsequent pairing
is the later pairing between the previously initial-paired device and
the other phone where the same Google Account is logged-in.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

phone -> device

@kapi-no kapi-no force-pushed the fast_pair_subsequent_pairing branch 6 times, most recently from be985bd to ded5d24 Compare July 2, 2024 10:52
@kapi-no kapi-no requested review from MarekPieta and zycz July 2, 2024 10:52
@MarekPieta MarekPieta removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Jul 2, 2024
@kapi-no
Copy link
Contributor Author

kapi-no commented Jul 2, 2024

@MarekPieta, please do not remove the changelog label. Before I can finalize this PR, I will need to create a changelog entry (once the release process is completed).

@kapi-no kapi-no force-pushed the fast_pair_subsequent_pairing branch from ded5d24 to c045969 Compare July 10, 2024 10:33
@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Jul 10, 2024
@kapi-no
Copy link
Contributor Author

kapi-no commented Jul 10, 2024

Rebased only. No file diff has changed.

@kapi-no kapi-no force-pushed the fast_pair_subsequent_pairing branch from c045969 to fd37c4c Compare July 10, 2024 10:46
@github-actions github-actions bot removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Jul 10, 2024
@kapi-no
Copy link
Contributor Author

kapi-no commented Jul 10, 2024

The last force-push add changelog entries for this modification.

@kapi-no kapi-no force-pushed the fast_pair_subsequent_pairing branch from fd37c4c to a61c946 Compare July 10, 2024 12:33
@kapi-no
Copy link
Contributor Author

kapi-no commented Jul 10, 2024

The last force-push adds a small fix for the build issue after I moved the validation logic to a different function following the review suggestion.

doc/nrf/external_comp/bt_fast_pair.rst Outdated Show resolved Hide resolved
doc/nrf/external_comp/bt_fast_pair.rst Outdated Show resolved Hide resolved
doc/nrf/external_comp/bt_fast_pair.rst Outdated Show resolved Hide resolved
doc/nrf/external_comp/bt_fast_pair.rst Outdated Show resolved Hide resolved
doc/nrf/external_comp/bt_fast_pair.rst Outdated Show resolved Hide resolved
subsys/bluetooth/services/fast_pair/Kconfig.fast_pair Outdated Show resolved Hide resolved
@kapi-no kapi-no force-pushed the fast_pair_subsequent_pairing branch from a61c946 to ad2fae8 Compare July 10, 2024 13:38
@kapi-no kapi-no requested a review from annwoj July 10, 2024 13:39
Improved the description of the bt_fast_pair_set_pairing_mode API
function in the "Setting up Bluetooth LE advertising" step of the
Fast Pair integration guide.

Signed-off-by: Kamil Piszczek <Kamil.Piszczek@nordicsemi.no>
Improved the description of the enabling policy for a few Kconfig
entries in the Fast Pair service documentation.

Signed-off-by: Kamil Piszczek <Kamil.Piszczek@nordicsemi.no>
Added a new Kconfig option for the Fast Pair Subsequent Pairing
feature. As a result, this previously supported feature is now
configurable and can be disabled for the locator tag use case,
which does not support subsequent pairing.

Aligned the Fast Pair Advertising Provider with the new Kconfig option
for the Fast Pair service, which controls the Subsequent Pairing
feature.

Removed the unavailable API call of the Fast Pair Advertising Provider
from the Fast Pair Locator Tag sample.

Ref: NCSDK-27955

Signed-off-by: Kamil Piszczek <Kamil.Piszczek@nordicsemi.no>
@kapi-no kapi-no force-pushed the fast_pair_subsequent_pairing branch from ad2fae8 to 133f1c1 Compare July 15, 2024 08:56
@kapi-no
Copy link
Contributor Author

kapi-no commented Jul 15, 2024

Rebased only due to the failing BLE CI. No file diff has changed.

@rlubos rlubos merged commit b32d531 into nrfconnect:main Jul 15, 2024
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc-required PR must not be merged without tech writer approval.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants