Skip to content
This repository has been archived by the owner on Dec 21, 2023. It is now read-only.

fix: url in drop screen #2325

Merged
merged 2 commits into from
Jul 31, 2023
Merged

fix: url in drop screen #2325

merged 2 commits into from
Jul 31, 2023

Conversation

intergalacticspacehighway
Copy link
Collaborator

Why

URL doesn't represent the currently selected drop sometimes.

@vercel
Copy link

vercel bot commented Jul 31, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
showtime ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 31, 2023 7:17pm

Copy link
Contributor

@hirbod hirbod left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One small issue I see, otherwise LGTM

packages/app/screens/swipe-list.tsx Outdated Show resolved Hide resolved
@github-actions
Copy link

github-actions bot commented Jul 31, 2023

PR Preview - App

This pull request preview deployment is now available.

Open Details

✅ Preview: exp+io.showtime.development://expo-development-client/?url=https://u.expo.dev/45cbf5d5-24fe-4aa6-9580-acf540651abd&channel-name=pr-2325

Comment ID: 1658920662

packages/app/screens/swipe-list.tsx Outdated Show resolved Hide resolved
@github-actions
Copy link

PR Preview - Storybook on Chromatic

✅ Preview: The Storybook preview URL for this Pull Request

✅ Build: The Chromatic build URL for this Pull Request

Copy link
Contributor

@hirbod hirbod left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@intergalacticspacehighway intergalacticspacehighway merged commit 9f33f7a into staging Jul 31, 2023
5 of 8 checks passed
@intergalacticspacehighway intergalacticspacehighway deleted the fix/url-nft-feed branch July 31, 2023 20:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants