-
-
Notifications
You must be signed in to change notification settings - Fork 360
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
✨ spotlight backend sorting #1913
✨ spotlight backend sorting #1913
Conversation
✔️ Deploy Preview for koda-nuxt ready! 🔨 Explore the source changes: 5f47055 🔍 Inspect the deploy log: https://app.netlify.com/sites/koda-nuxt/deploys/61e7c17269f63200078acf60 😎 Browse the preview: https://deploy-preview-1913--koda-nuxt.netlify.app |
Is it related? kodadot/rubick#23 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👀
@Prop(String) public account!: string | ||
protected nfts: NftSimpleView[] = [] | ||
protected isLoading = true | ||
|
||
protected async fetchNFT(account: string) { | ||
const nfts = await this.$apollo.query({ | ||
query: nftSimpleListByAccount, | ||
client: this.urlPrefix, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
spotlight has nothing to do with prefixmixin
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@vikiival it throw this without client
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
client: this.urlPrefix, | |
client: 'subsquid', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not subsquid, rmrk
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pushed some changes
This is related to #1893 but yes we can try the same with spotlight resolver. |
shiiiiit |
/giphy shit |
I'm not able to sort on averagePrice column yet.
PR type
What's new?
Before submitting Pull Request, please make sure: