Conversation
Reduces exposure to supply chain attacks via freshly published malicious versions by filtering out npm releases younger than 3 days during install. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…ction chore: add 3-day minimum release age for npm packages
Render the offers section on the gift cards page differently based on offer count: 1 offer stays full-sized, 2 offers go side-by-side at half width, 3+ become a horizontal scroll carousel at discover-card size. Extract the discover section's scroll-position-restoration hook into a shared useRestoreScrollPosition keyed by location.state name, used by both DiscoverGiftCards and the new OffersSection.
Merge existing location.state in getScrollState so Discover and Offers sections forward-carry each other's scroll position on navigation. Drop the includeScrollState boolean in OffersSection so the 1/2-offer paths no longer wipe discoverScrollPosition. Inline the zod schema and remove the useMemo per review feedback.
ui: use close button on scanner routes
Offers icon should be Gift, not Bank icon
Solid page header on /gift-cards
The merge in getScrollState forwarded every field of the current
location.state into the next navigation — including the view-transition
{transition, applyTo} fields stored by LinkWithViewTransition. That
caused offer/discover navigations from /gift-cards (raw navigate, no
transition) to inherit the home→/gift-cards slide direction, producing
wrong-direction slide animations after a hard reset.
Strip the view-transition fields from the merge so only caller-owned
state (e.g. sibling scroll positions) is forwarded.
…leak Prevent view-transition state leak via scroll restoration
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
bobscully3
approved these changes
May 14, 2026
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
5 participants
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.
No description provided.