-
Notifications
You must be signed in to change notification settings - Fork 59
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
feat: nft metadata service #769
Conversation
32c63a4
to
46185ad
Compare
f4c3d50
to
f138615
Compare
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.
Looks great @mkazlauskas! 🚀
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.
Great work @mkazlauskas, can squash, but let's hold on merging until the #634
42f3708
to
191eb8f
Compare
ebfb09d
to
b58aa9d
Compare
df729b0
to
05fb0cc
Compare
05fb0cc
to
9709fc7
Compare
4f4af9d
to
a85b1b0
Compare
f61cd46
to
c078a91
Compare
a8b98b2
to
c078a91
Compare
also clean up the test by removing duplicate data
there can be multiple 'copies' of nft metadata for an asset in a block it was keeping the 1st one instead
hoist from NftMetadata.fromPlutusData and add tests
it maps 'og' from base nft metadata and images from 'extra'
…mentations - hoist assetNameToUTF8Handle to util.ts - use AssetName.toUTF8 core util also, replace custom error with the existing InvalidStringError, because it wasn't used anywhere
hoist shared utils to util.ts
…rPaymentCredential columns
also add missing entity inter-dependency output->tokens
it would omit the prop if no stake pools of some status are in the database
in reality those shouldn't be in user's wallet, but technically they can be
to be used when generating cardano-services db snapshot
add a cip68 handle to handle test db disable some flaky tests: there are no 'retiring' pools in db fix TypeormStakePoolProvider test setup to match partial strings
address could also be owned by script
…prop it does not add any new columns, but simplifies queries
in some cases it returned a Buffer, because Buffer.valueOf() => Buffer in nodejs
Buffer is Node.js-specific ArrayBuffer and it's views is available in web environments as well
4be97d5
to
78460d3
Compare
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.
LGTM 🚀
FEATURE BRANCH
In addition to Context below, it has:
Context for original scope
feat: add PlutusData core types
PlutusData (datums & redeemers) are currently typed as HexBlob. We need to create better types in order to be able to parse/interpret and build them.
Breaking changes
commit hashes below are outdated
9c39bbd- this is a major breaking change. Provider server is backward and forward incompatible.e024c9ecb6bc4e