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

🦺 🐛 syntax error at or near "undefined" #268

Merged
merged 2 commits into from
Aug 11, 2023
Merged

🦺 🐛 syntax error at or near "undefined" #268

merged 2 commits into from
Aug 11, 2023

Conversation

vikiival
Copy link
Member

  • 🐛 [MINT] syntax error at or near "undefined"
  • ⬆️ @kodadot1

Thank you for your contribution to the KodaDot Indexer.
👇 _ Let's make a quick check before the contribution.

PR type

  • Bugfix
  • Feature
  • Refactoring

What's new?

I got failing prod on rubick v9 and started to debug it.

The failing point was that optional argument (the one that can be undefined) was included into query and PSQL does not KNOW undefined
This debug cost me an ~90min :)

[MINT] syntax error at or near "undefined" {"collection":"dac5c7f54029d0e73c-50JVJ","name":"TR80085-4/CH","instance":"TR800854CH","transferable":1,"sn":"0000000000000001","metadata":"ipfs://ipfs/bafkreig7nl2wht6tinguyt6lmuer7uk6wqnq67cdyo3z6suivgedgynhoq"}
11.08.2023 13:33:30
Error
db
syntax error at or near "undefined" SELECT COUNT(DISTINCT current_owner) AS distribution, COUNT(current_owner) AS owner_count ,(SELECT max(CASE WHEN current_owner = 'HXAi8NoJHQUU1FQUJrHYGtAECN1upNKgPPnJMj7er1eFRK8' THEN 0 ELSE 1 END) FROM nft_entity) AS adjustment FROM nft_entity WHERE collection_id = 'dac5c7f54029d0e73c-50JVJ' undefined

@vikiival vikiival changed the title RMRK One 🦺 🐛 syntax error at or near "undefined" Aug 11, 2023
@socket-security
Copy link

Updated dependencies detected. Learn more about Socket for GitHub ↗︎

Packages Version New capabilities Transitives Size Publisher
@kodadot1/minimark 0.1.9-rc.0...0.1.10 None +41/-41 53.3 MB vikiival
@kodadot1/minipfs 0.3.0-rc.0...0.4.0-rc.0 environment +3/-4 745 kB vikiival

@vikiival vikiival merged commit 2fa4c72 into main Aug 11, 2023
@vikiival vikiival deleted the rmrk-one branch August 11, 2023 10:46
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.

1 participant