Skip to content

Add a browsable top rated movies resource - #54

Merged
stphnwlsh merged 15 commits into
mainfrom
feature/top-rated-resource
Nov 6, 2026
Merged

stphnwlsh merged 15 commits into
mainfrom
feature/top-rated-resource

Conversation

@stphnwlsh

Copy link
Copy Markdown
Collaborator

Summary

Adds movies://top-rated, a static resource with the ten best rated movies, so the Resources tab has something to open without knowing an id. It uses the existing movies query and ranks in the resource, so no Application code changes.

This is stacked on #53 for the icons, and will retarget to main when #53 merges.

Related issues

None.

Type of change

  • Bug fix
  • New feature
  • Breaking change (the generated solution or template parameters change in a way existing users will notice)
  • Refactoring or clean-up
  • Documentation
  • Build, pipeline or dependency update

Shared layers

  • This only touches the Presentation layer of this template.
  • This touches the Application or Infrastructure layers, tests, pipeline or repository setup, so it needs to be carried across all five templates.

Checklist

  • dotnet build and dotnet test pass locally with no warnings.
  • I added or updated tests, and line coverage stays at or above 95%.
  • I updated the README or docs where behaviour changed.
  • If this changes how we do something, I updated the matching skill in .agents/skills.
  • I checked the template still installs and creates a working solution (see CONTRIBUTING.md).

@stphnwlsh stphnwlsh self-assigned this Nov 6, 2026
@github-actions github-actions Bot added the enhancement New feature or request label Nov 6, 2026
Base automatically changed from feature/completions-and-icons to main November 6, 2026 19:01
@github-actions

github-actions Bot commented Nov 6, 2026

Copy link
Copy Markdown
Contributor

Build Pipeline

✅ Ready to merge. Every stage passed.

Stage Result Details
Build 🟢 Passed Version 2026.11.06.203-pr-54 · Template package 240 KB
Test 🟢 Passed 275 passed · 0 failed · 0 skipped · Template installs, builds and passes its tests
Coverage 🟢 Passed 100% (933 of 933 lines) · Needs 95%
Static Analysis 🟢 Passed No open CodeQL alerts
Dependency Review 🟢 Passed 0 vulnerable packages · 0 licence problems
Container Scan 🟢 Passed Image starts and answers · No open Trivy alerts

Commit 4569a0f · Full run

@stphnwlsh
stphnwlsh merged commit f99845f into main Nov 6, 2026
13 checks passed
@stphnwlsh
stphnwlsh deleted the feature/top-rated-resource branch November 6, 2026 19:32
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.

1 participant