Skip to content

Conversation

@mirkoCrobu
Copy link
Contributor

Motivation

closes #114

The v1/apps/{id}/bricks response must not include compatible_models fields.

Change description

Decouple the v1/apps/{id}/bricks and v1/apps/{id}/bricks/{brick_id} response structures by creating a dedicated struct for the v1/apps/{id}/bricks endpoint.

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 requested a review from dido18 November 27, 2025 17:07
@mirkoCrobu mirkoCrobu self-assigned this Nov 27, 2025
@mirkoCrobu mirkoCrobu added bug Something isn't working enhancement New feature or request labels Nov 27, 2025
@mirkoCrobu mirkoCrobu merged commit 3f54506 into main Nov 28, 2025
11 checks passed
@mirkoCrobu mirkoCrobu deleted the issue_114_rm_compatible_modesl_from_instance_brick_list_2 branch November 28, 2025 07:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove the compatible_models from the /v1/apps/{id}/bricks

2 participants