⚠️ do not merge/arrowsquid ahp

[vikiival]

• 🔧 arrow stick AHP

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.

👇 __ Let's make a quick check before the contribution.

PR Type

• Bugfix
• Feature
• Refactoring

Needs QA check

• @kodadot/qa-guild please review

Context

• Closes #<issue_number>
• Requires deployment <snek/rubick/worker>

Did your issue had any of the "$" label on it?

• My DOT address: Payout

Screenshot 📸

• My fix has changed UI

[netlify]

✅ Deploy Preview for koda-canary ready!

Name	Link
🔨 Latest commit	e73dc2a
🔍 Latest deploy log	https://app.netlify.com/sites/koda-canary/deploys/65f29bc3f5b736000876a1b9
😎 Deploy Preview	https://deploy-preview-9577--koda-canary.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

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

[yangwao]

@vikiival, what are you waiting for here?

[vikiival]

• update or delete on table "token_entity" violates foreign key constraint stick#220 (comment)

Made another release

[vikiival]

If it is OK let's close it then

[prury]

@vikiival i'm only getting some console errors when accessing galleryitem, but everyhing is loading regardless

[vikiival]

Ah I forgot to enable ws, thanks for the test

[vikiival]

issue was in the trailing slash e73dc2a

[codeclimate]

Code Climate has analyzed commit e73dc2a and detected 0 issues on this pull request.

View more on Code Climate.

[sonarcloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

[yangwao]

Needs to be tested and then can be promoted to production?

[vikiival]

Needs to be tested and then can be promoted to production?

Exactly, once we close this PR i will promote arrowsquid to prod

[prury]

@vikiival i'm getting different query results:

[vikiival]

@vikiival i'm getting different query results:

I have missed the PR from @daiagi

• 🚑 Token Entity burn stick#251

Should be fixed in ~30 min

[prury]

@vikiival

[vikiival]

Probably fixed ?

kudos to @daiagi

[yangwao]

putting it live?

[prury]

Probably fixed ?

kudos to @daiagi

yup, this is fixed

only small difference in query results now (21) items on explore -> items

[vikiival]

only small difference in query results now (21) items on explore -> items

I am not sure what you mean. Can you provide screenshots please?

[prury]

@vikiival i'm getting different query results:

Same as this, but with smaller difference

[vikiival]

Same as this, but with smaller difference

I suppect there is a huge hidden bug in the token entity (cc @daiagi)

I am wondering what should be the "correct" state tbh

• Run comparison tests stick#255

[daiagi]

Should be some sql issue in the total token entity resolver, nothing major (in terms of effort to fix)
Can assume that previously number is the correct one as I checked it well when I first wrote it. And the previous query was simple

[daiagi]

Something completely different is going on here and it's not so easy to explain over gh post

1. it isn't related to tokenEntity at all, and token entity is reporting correct numbers (compare PR and canary with "the break" search query)

2. it's about metadata mismatch between canary and pr env. no idea why. maybe because the indexer is stuck?

let's walk through it, we will focus on Motifs collection

1. in explore page, search for "motifs"
2. notice that count@pr = count@canary +4
3. scroll to bottom and find a stacked nft with countof 8@canary and count of 4@PR. this account for all the mismatch
4. go to collection page (of motifs), collection page does not use TokenEntity at all, notice that total count is same in both envs
5. manually count the amount of stacked nfts, you will find that the count on the "stacked" card was individually correct in each env
6. think to yourself wtf?
7. find out that nft 108-88, shows the default drop picture on canary (stacked) and NOT the default picture on PR env (not stacked)

tl;dr

some nfts have default image on canary and not default on PR env, which leads to a different count in explore page due to "stacked" cards @vikiival ant thoughts? my only idea is due to stuck indexer

also, canary env does not impl yet the "tokenEntity" id change, that fixed "tokenEntity" bunching together nfts that should not be bunched together, that also leads to count mismatch

[vikiival]

also, canary env does not impl yet the "tokenEntity" id change, that fixed "tokenEntity" bunching together nfts that should not be bunched together, that also leads to count mismatc

Except this line I was able to understand the issue you have described above ^-^, thanks for the explanation.

probably this would be a correct thing to do

• Run comparison tests stick#255

[vikiival]

So @subsquid turned off the prod endpoint, so I will let this sync and promote it to prod

[vikiival]

[vikiival]

Side effects are my favs