Skip to content

index: Add nrfcloud quickstart bluetooth v1.1.0 - #133

Merged
thst-nordic merged 1 commit into
nrfconnect:mainfrom
noahp:noahp/quickstart-bluetooth-v1.0.1
Aug 24, 2026
Merged

thst-nordic merged 1 commit into
nrfconnect:mainfrom
noahp:noahp/quickstart-bluetooth-v1.0.1

Conversation

@noahp

@noahp noahp commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@noahp
noahp requested a review from a team as a code owner August 6, 2026 12:04
Comment thread index/nrfconnect.json Outdated
"description": "The peripheral_lbs button/LED-over-BLE sample plus Memfault observability (heartbeat metrics and coredumps) delivered over the Memfault Diagnostic Service BLE gateway path, for the nRF Connect for Desktop Quick Start guide. Targets the nRF54L15 DK only.",
"kind": "sample",
"tags": ["bluetooth", "ble", "connectivity"],
"apps": "project/app",

@kylebonnici kylebonnici Aug 6, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Also others have app and not project/app ... not sure why to be honest it is unclear to me which is the correct value

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It's from how this project is configured, I don't know why the original author set it up that way-
https://github.com/nrfconnect/quickstart-bluetooth/blob/main/west.yml#L23
nested under project/app 🤷

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

from what i can tell, this is just informative today- i think the asset tracker one is possibly mismatched, but i don't know 🤷 maybe a maintainer can comment, @thst-nordic ?

@thst-nordic thst-nordic Aug 24, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

"apps" is passed to vscode extension presumably so it can search the folder for apps to list as buildable.

I found out vscode isn't using it. so we will do a cleanup round on all projects when they start using it.
no changes needed.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

updated, thanks!

@noahp

noahp commented Aug 10, 2026

Copy link
Copy Markdown
Contributor Author

@jangalda-nsc this is ready for you 🙏

@gregersrygg

Copy link
Copy Markdown

@noahp do we want to update this PR to v1.1.0?

Comment thread index/nrfconnect.json Outdated
@noahp

noahp commented Aug 21, 2026

Copy link
Copy Markdown
Contributor Author

@noahp do we want to update this PR to v1.1.0?

Yep, might as well 😄

@noahp noahp changed the title index: Add nrfcloud quickstart bluetooth v1.0.1 index: Add nrfcloud quickstart bluetooth v1.1.0 Aug 21, 2026
@thst-nordic

Copy link
Copy Markdown
Contributor

i'll check on project/app.
squash to 1 commit and Its good to merge.

Comment thread index/nrfconnect.json Outdated
"description": "The peripheral_lbs button/LED-over-BLE sample plus Memfault observability (heartbeat metrics and coredumps) delivered over the Memfault Diagnostic Service BLE gateway path, for the nRF Connect for Desktop Quick Start guide. Targets the nRF54L15 DK only.",
"kind": "sample",
"tags": ["bluetooth", "ble", "connectivity"],
"apps": "project/app",

@thst-nordic thst-nordic Aug 24, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

"apps" is passed to vscode extension presumably so it can search the folder for apps to list as buildable.

I found out vscode isn't using it. so we will do a cleanup round on all projects when they start using it.
no changes needed.

@noahp
noahp force-pushed the noahp/quickstart-bluetooth-v1.0.1 branch from d19fbd5 to d4ed6fb Compare August 24, 2026 12:19
@noahp
noahp requested a review from thst-nordic August 24, 2026 12:19
@thst-nordic
thst-nordic added this pull request to the merge queue Aug 24, 2026
Merged via the queue into nrfconnect:main with commit af65851 Aug 24, 2026
3 checks passed
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