-
Notifications
You must be signed in to change notification settings - Fork 85
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: updates Polkadot.js packages #830
Conversation
Visit the preview URL for this PR (updated for commit 7335789): https://astar-apps--pr830-feat-update-polkadot-2fpbbn2k.web.app (expires Tue, 27 Jun 2023 02:34:06 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: dd76fe72958fe2910fef9d53f0b4539b82b849db |
when I debug this, regarding |
@sirius651 I think that Polkadot.js packages require importing BN from '@polkadot/util' package (as @bobo-k2 explained in slack) but you can try. |
|
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.
The tests run well, the code looks good.
Pull Request Summary
Check list
This pull request makes the following changes:
Changes
To-dos
[Resolved] Resolve unknown errors
Resolve
test
(yarn test)