Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

API: add "UsedByApps" to the details of a brick#30

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
mirkoCrobu merged 8 commits intomainfromissue_807
Nov 3, 2025

Conversation

@lucarin91
Copy link
Contributor

@lucarin91lucarin91 commentedOct 28, 2025•
edited by dido18
Loading

Motivation

The API returning the details of a brick show fill in theUsedByApps 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 inmain.
  • Changes are covered by tests.
  • Logging is meaningful in case of troubleshooting.

@mirkoCrobumirkoCrobu changed the titleIssue 807API: add "UsedByApps" to the details of a brickOct 29, 2025
@mirkoCrobumirkoCrobu self-assigned thisOct 29, 2025
@mirkoCrobumirkoCrobu marked this pull request as ready for reviewOctober 30, 2025 08:19
@mirkoCrobumirkoCrobu requested a review froma teamOctober 30, 2025 08:20
@lucarin91lucarin91 added the enhancementNew feature or request labelOct 30, 2025
Copy link
ContributorAuthor

@lucarin91lucarin91 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

mirkoCrobu reacted with rocket emoji
@CLAassistant
Copy link

CLAassistant commentedNov 3, 2025•
edited
Loading

CLA assistant check
All committers have signed the CLA.

@mirkoCrobumirkoCrobu merged commit1b200b1 intomainNov 3, 2025
9 checks passed
@mirkoCrobumirkoCrobu deleted the issue_807 branchNovember 3, 2025 17:18
mirkoCrobu added a commit that referenced this pull requestNov 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 freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@dido18dido18dido18 left review comments

@mirkoCrobumirkoCrobumirkoCrobu left review comments

@XaytonXaytonXayton approved these changes

Assignees

@mirkoCrobumirkoCrobu

Labels

enhancementNew feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@lucarin91@CLAassistant@dido18@Xayton@mirkoCrobu
[8]ページ先頭
©2009-2025 Movatter.jp