Skip to content

Conversation

@divipillai
Copy link
Contributor

Peripheral_lbs sample:

  • A screenshot of how to enable notifications (or at least explain the icon: downward arrow with bottom line)
  • A screenshot of writing value (up arrow) + showing "Bool" selected

NCSDK-35994

@NordicBuilder NordicBuilder 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 Oct 27, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Oct 27, 2025

CI Information

To view the history of this post, click the 'edited' button above
Build number: 4

Inputs:

Sources:

more details

Github labels

Enabled Name Description
ci-disabled Disable the ci execution
ci-all-test Run all of ci, no test spec filtering will be done
ci-force-downstream Force execution of downstream even if twister fails
ci-run-twister Force run twister
ci-run-zephyr-twister Force run zephyr twister
List of changed files detected by CI (0)

Outputs:

Toolchain

Version:
Build docker image:

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

  • ◻️ Toolchain
  • ◻️ Build twister
  • ◻️ Integration tests

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

@divipillai divipillai force-pushed the NCSDK-35994_peripheral_lbs_image branch from ce64e97 to 88c95cb Compare October 27, 2025 14:06
@divipillai divipillai added CI-disable Disable CI for this PR and removed changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Oct 27, 2025
@divipillai divipillai force-pushed the NCSDK-35994_peripheral_lbs_image branch from 88c95cb to 1684002 Compare October 27, 2025 15:42
@NordicBuilder NordicBuilder added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Oct 27, 2025
@divipillai divipillai removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Oct 27, 2025
@divipillai divipillai marked this pull request as ready for review October 27, 2025 16:21
@divipillai divipillai requested review from a team as code owners October 27, 2025 16:21
@github-actions
Copy link

You can find the documentation preview for this PR here.

Preview links for modified nRF Connect SDK documents:

https://ncsdoc.z6.web.core.windows.net/PR-25257/nrf/samples/bluetooth/peripheral_lbs/README.html

Peripheral_lbs sample:

* A screenshot of how to enable notifications
(or at least explain the icon: downward arrow with bottom line)
* A screenshot of writing value (up arrow) + showing "Bool" selected

NCSDK-35994

Signed-off-by: divya pillai <[email protected]>
@divipillai divipillai force-pushed the NCSDK-35994_peripheral_lbs_image branch from 1684002 to bde44cb Compare October 27, 2025 16:32
@NordicBuilder NordicBuilder added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Oct 27, 2025
@NordicBuilder NordicBuilder requested review from a team October 27, 2025 16:32
Copy link

@KievDevel KievDevel left a comment

Choose a reason for hiding this comment

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

LGTM

@rlubos rlubos merged commit 2cce3b7 into nrfconnect:main Oct 28, 2025
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. CI-disable Disable CI for this PR doc-required PR must not be merged without tech writer approval.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants