Skip to content

Conversation

@lucarin91
Copy link
Contributor

@lucarin91 lucarin91 commented Oct 28, 2025

Motivation

The API returning the details of a brick show fill in the UsedByApps property.
The App Lab uses this to show what examples or apps are using a given brick.

Change description

Additional Notes

Reviewer checklist

  • PR addresses a single concern.
  • PR title and description are properly filled.
  • Changes will be merged in main.
  • Changes are covered by tests.
  • Logging is meaningful in case of troubleshooting.

@mirkoCrobu mirkoCrobu changed the title Issue 807 API: add "UsedByApps" to the details of a brick Oct 29, 2025
@mirkoCrobu mirkoCrobu self-assigned this Oct 29, 2025
@mirkoCrobu mirkoCrobu marked this pull request as ready for review October 30, 2025 08:19
@mirkoCrobu mirkoCrobu requested a review from a team October 30, 2025 08:20
@lucarin91 lucarin91 added the enhancement New feature or request label Oct 30, 2025
Copy link
Contributor Author

@lucarin91 lucarin91 left a comment

Choose a reason for hiding this comment

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

LGTM, just a few small changes

@CLAassistant
Copy link

CLAassistant commented Nov 3, 2025

CLA assistant check
All committers have signed the CLA.

@mirkoCrobu mirkoCrobu merged commit 1b200b1 into main Nov 3, 2025
9 checks passed
@mirkoCrobu mirkoCrobu deleted the issue_807 branch November 3, 2025 17:18
mirkoCrobu added a commit that referenced this pull request Nov 4, 2025
The API returning the details of a brick show fill in the UsedByApps property.
The App Lab uses this to show what examples or apps are using a given brick.* add useByApps field for brick details endpoint

* partial test implementation

* add test end2end

* delete wrong tests

* refactoring

* make lint happy

* code review fixes

* fix error message

---------

Co-authored-by: mirkoCrobu <m.crobu@ext.arduino.cc>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants