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

fix: showing onboarding modal on collecting #2384

Merged

Conversation

intergalacticspacehighway
Copy link
Collaborator

Why

How

Test Plan

@vercel
Copy link

vercel bot commented Aug 18, 2023

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

Name Status Preview Updated (UTC)
showtime 🔄 Building (Inspect) Visit Preview Aug 18, 2023 7:58pm

@@ -29,19 +32,27 @@ const { useParam } = createParam<{
export const CheckoutReturnForPaidNFT = () => {
const [contractAddress] = useParam("contractAddress");
const { data: edition } = useCreatorCollectionDetail(contractAddress);
if (!edition)
const { data: nft } = useNFTDetailByTokenId({
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

had to fetch the nft as the latest_star field needs some values from it.

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 bf93996 into staging Aug 18, 2023
4 of 6 checks passed
@intergalacticspacehighway intergalacticspacehighway deleted the fix/onboarding-modal-on-collecting branch August 18, 2023 20:03
@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

@github-actions
Copy link

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-2384

Comment ID:

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.

2 participants