Skip to content

[codex] View movie details - #28

Merged
marciomyst merged 2 commits into
developfrom
013-view-movie-details
May 9, 2026
Merged

[codex] View movie details#28
marciomyst merged 2 commits into
developfrom
013-view-movie-details

Conversation

@marciomyst

Copy link
Copy Markdown
Owner

What changed

Implements movie details end to end for issue 13.

  • Adds the backend GET /api/movies/{id} endpoint with typed success and ProblemDetails responses.
  • Adds application and repository support for loading a movie by ID with genres.
  • Adds the public movie details contract using posterUrl.
  • Adds Angular typed service/model support and the /movies/:id details page.
  • Makes movie cards navigate to the details route.
  • Updates Spec Kit artifacts and documentation for the feature.
  • Aligns the Angular project location in the Visual Studio solution file.

Impact

Users can click a movie card in the catalog and view a dedicated details screen with poster, metadata, synopsis, and handled loading/error/not-found states.

Validation

  • dotnet build SmartMovieCatalog.slnx
  • Earlier in the branch: backend tests, frontend build, and frontend tests were run while implementing the feature.

Notes

The latest solution build succeeded with warnings from npm package audit and transient Visual Studio/API file locks, but no build errors.

@marciomyst marciomyst linked an issue May 9, 2026 that may be closed by this pull request
@sonarqubecloud

sonarqubecloud Bot commented May 9, 2026

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
79.5% Coverage on New Code (required ≥ 80%)
D Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@marciomyst
marciomyst marked this pull request as ready for review May 9, 2026 10:02
@marciomyst
marciomyst merged commit 968992a into develop May 9, 2026
3 of 6 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.

View movie details

1 participant