-
Notifications
You must be signed in to change notification settings - Fork 60
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: browser compatible BlockfrostAssetProvider #1528
feat: browser compatible BlockfrostAssetProvider #1528
Conversation
dab01bb
to
6e23df9
Compare
|
packages/cardano-services/src/util/BlockfrostProvider/BlockfrostClientFactory.ts
Show resolved
Hide resolved
60d1190
to
aef51d8
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.
Fantastic work @mkazlauskas 🚀
replace BlockfrostAssetProvider in cardano-services with a new one that works in the browser implementation was hoisted from 'lace' repo BREAKING CHANGE: hoist BlockfrostAssetProvider to cardano-services-client
… from array to string also log nft metadata mapping errors
…cified in metadata
2e52466
to
b5c9f26
Compare
Context
BlockfrostAssetProvider only works in nodejs
Proposed Solution
replace BlockfrostAssetProvider in cardano-services with a new one that works in the browser
implementation was hoisted from 'lace' repo
Important Changes Introduced
BREAKING CHANGE: hoist BlockfrostAssetProvider to cardano-services-client