Skip to content

fixed no account nfts loading state#58

Merged
poocart merged 2 commits intostagingfrom
fix/fixed-acc-nfts-loading
Mar 1, 2024
Merged

fixed no account nfts loading state#58
poocart merged 2 commits intostagingfrom
fix/fixed-acc-nfts-loading

Conversation

@poocart
Copy link
Contributor

@poocart poocart commented Mar 1, 2024

Description

  • Fixed stuck loading state on Account bottom modal NFTs

How Has This Been Tested?

  • Local dapp, tests.

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

@poocart poocart requested a review from IAmKio March 1, 2024 09:39
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Mar 1, 2024

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 6b3c4e3
Status:⚡️  Build in progress...

View logs

"pageNotFound": "Page not found",
"appNotFound": "App not found",
"transactionFailedReasonUnknown": "Transaction failed: reason unknown"
"transactionFailedReasonUnknown": "Transaction failed: reason unknown",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Just a note for the future - we'll change this to give the user an instruction on how to try to continue

Copy link
Collaborator

@IAmKio IAmKio left a comment

Choose a reason for hiding this comment

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

LGTM

Base automatically changed from chore/ui-cleanup to staging March 1, 2024 11:26
@github-actions github-actions bot temporarily deployed to Preview (staging) March 1, 2024 11:30 Inactive
@poocart poocart merged commit de705bd into staging Mar 1, 2024
@poocart poocart deleted the fix/fixed-acc-nfts-loading branch March 1, 2024 11:30
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