Skip to content

feat: make recipe cards interactive with detail page and Run Match - #353

Merged
touchthesun merged 1 commit into
mainfrom
feat/recipe-card-navigation
Aug 12, 2026
Merged

feat: make recipe cards interactive with detail page and Run Match#353
touchthesun merged 1 commit into
mainfrom
feat/recipe-card-navigation

Conversation

@touchthesun

Copy link
Copy Markdown
Contributor

Summary

  • Recipe cards on the cooking-domain browse page were static text — no way to open a recipe or run a match from the list, unlike design cards.
  • Add RecipeDetailView (mirroring OkhDetailView's breadcrumb/title/actions layout) rendered at the existing okh/:id route when domain=cooking, reusing the already-cached ["recipes"] query since there's no per-recipe fetch endpoint.
  • Link each recipe card's body to its detail page, and add a "Run Match ⚡" action on both the card and the detail page that navigates to /match?recipe_id=<id>.
  • MatchPage/CookingMatchView now read recipe_id from the URL and preselect it in RecipePicker, mirroring how okh_id preselects a design on the hardware match page.

Test plan

  • npm run typecheck
  • npm run lint (no new warnings/errors on touched files)
  • npm run test:unit — 346/346 passing, including new tests for RecipeListView, RecipeDetailView, and CookingMatchView's initialRecipeId preselect
  • Manually click through a recipe card → detail page → Run Match on a deployed cooking-domain instance

Made with Cursor

Recipe cards were static text with no way to view a recipe or match it,
unlike design cards. Add a RecipeDetailView (mirroring OkhDetailView) at
the existing okh/:id route for the cooking domain, link each recipe card
to it, and give both the card and the detail page a Run Match action that
preselects the recipe on the match page via ?recipe_id=.

Co-authored-by: Cursor <cursoragent@cursor.com>
@touchthesun
touchthesun merged commit 18c708e into main Aug 12, 2026
7 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.

2 participants