"You might also like" on the recipe preview screen - #77
Merged
Conversation
A recipe opened from an outside link lands on the preview, which showed no similar recipes — the section only existed on the saved detail screen. Add a horizontal "You might also like" strip at the bottom of the preview, powered by GET /v1/recipes/similar-by-url (api companion): cards of real recipes from the extraction pool similar to the previewed page. Tapping one opens that recipe's own preview. The strip quietly hides while loading, on error, or when empty, so it never clutters a preview. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Companion to windoze95/saltybytes#142.
Why
User report: opening a recipe from an outside link (shared/universal link) shows no similar recipes at the bottom. Those links land on the preview screen, which had no such section (it only existed on the saved detail screen).
What
A horizontal "You might also like" strip at the bottom of the preview, powered by
GET /v1/recipes/similar-by-url(source URL → similar recipes from the extraction pool). Cards show the recipe title + source domain; tapping one opens that recipe's own preview (/preview?u=…, pushed so back returns here). The strip renders nothing while loading, on error, or when empty — so it never adds clutter to a preview the user is deciding whether to import.Also benefits the normal search → preview flow, not just deep links.
Tests
similarByUrlProvider+SimilarWebRecipemodel; 2 new preview-screen widget tests (populated strip + tap opens that preview; hidden when none). Existing preview tests get a default empty-similars stub. Full suite 472 green; analyze clean.🤖 Generated with Claude Code