Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Empty swap table with valid swaps #11299

Closed
wants to merge 12 commits into from

Conversation

hassnian
Copy link
Contributor

@hassnian hassnian commented Dec 26, 2024

PR Type

  • Bugfix
  • Feature
  • Refactoring

Context

Adds the ability to pass variables as ref to useGraphql

reviewers kindly also test , tyvm

Screenshot 📸

  • My fix has changed something on UI;

CleanShot 2024-12-26 at 13 23 40

Copy link

netlify bot commented Dec 26, 2024

Deploy Preview for koda-canary ready!

Name Link
🔨 Latest commit 8a5b396
🔍 Latest deploy log https://app.netlify.com/sites/koda-canary/deploys/6770fab201658c00081a8564
😎 Deploy Preview https://deploy-preview-11299--koda-canary.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@hassnian hassnian marked this pull request as ready for review December 26, 2024 08:29
@hassnian hassnian requested a review from a team as a code owner December 26, 2024 08:29
@hassnian hassnian marked this pull request as draft December 26, 2024 08:37
- add: cloneRawObject
- add: override param
- allow `useGrapqh` to be used outside setup , fix [Graphql Subscription] Polling error: Error: vue-query hooks can only be used inside setup() function or functions that support injection context.
- removed: mapToId util
- removed: existingCollectionList query
- simple solution to ensure only the latest query result updates the state.
@hassnian hassnian marked this pull request as ready for review December 27, 2024 11:29
@hassnian
Copy link
Contributor Author

@preschian any idea why the preview link is not working?

CleanShot 2024-12-28 at 10 44 40@2x

@exezbcz
Copy link
Member

exezbcz commented Dec 29, 2024

is not showing the skeleton first better and then fallback to the empty message?

@preschian
Copy link
Member

@preschian any idea why the preview link is not working?

hhmm, this is new for me. seems like need to check it through netlify dashboard cc @vikiival

Copy link
Member

@preschian preschian left a comment

Choose a reason for hiding this comment

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

Anyway, are you able to create a branch directly in kodadot/nft-gallery instead of branching from a fork?

This would allow us to see the deployment preview through Cloudflare Pages, since currently we're not able to see the deployment preview from Netlify. At the moment, Cloudflare Pages doesn't support deployment previews from forked branches

...extraVariables,
},
clientId: isRef(client) ? String(client.value) : client,
const requestId = ++currentRequestId.value
Copy link
Member

Choose a reason for hiding this comment

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

We probably don't need the useGraphql composable at all. I prefer using useQuery directly from NuxtApollo instead - that composable alone might be sufficient

Copy link
Contributor Author

@hassnian hassnian Dec 30, 2024

Choose a reason for hiding this comment

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

Yep, I like this solution a lot better. new pr #11303

import { mapToId } from '@/utils/mappers'
import { unwrapSafe } from '@/utils/uniquery'

export function useNewCollectionId() {
Copy link
Member

Choose a reason for hiding this comment

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

nit: let's move to separate PR regarding remove unused stuff

Copy link
Contributor Author

Choose a reason for hiding this comment

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

sure

@hassnian
Copy link
Contributor Author

closing in favour of #11303

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.

Empty swap table with valid swaps
3 participants