-
-
Notifications
You must be signed in to change notification settings - Fork 347
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
chore: update subapi package without warnings 📦 #4060
Conversation
✅ Deploy Preview for koda-nuxt ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
built and tested without problem 👍🏻 |
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.
wfm
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.
✅ Niiiiiice! did you spend more time than expected on this task?
debugging the pnpm problem was quite a hassle... 😓 (going through all the dependency trees and stuff like that) |
as far as I can tell this is gtg, please let's just resolve conflicts (sorry, merged some packages updates prematurely) @kkukelka 🙏🏻 |
efc8dc6
to
7737be7
Compare
done |
I'd be in favor of increasing the bounty on this PR maybe add some bonus for the time spent |
Code Climate has analyzed commit 4476bf0 and detected 0 issues on this pull request. View more on Code Climate. |
pay 125 usd |
@yangwao looks like hit or miss with the payouts for the past couple of days, could you check when you find time? |
😍 Perfect, I’ve sent the payout 🪅 Let’s grab another issue and get rewarded! |
Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR type
What's new?
due to pnpm's inability to resolve this correctly in every case, we have to manually do an override for
@polkadot/utils
you can read more about the issue here
Ensure/check single version of a dependency package throughout workspace pnpm/pnpm#2713
Support yarn’s resolutions package.json field pnpm/pnpm#1221
this also fixed transaction fees at simple mint
Before submitting Pull Request, please make sure:
Optional
Had issue bounty label?
Community participation
Screenshot