-
Notifications
You must be signed in to change notification settings - Fork 3
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
Update/iotex antenna #263
Update/iotex antenna #263
Conversation
@ccali11 please remove it from the crawler package as well. |
You got it, @hawyar 👍 |
@hawyar - what would you like me to do with |
Feel free to remove the file |
And all traces of |
services/crawler/package.json
Outdated
@@ -11,8 +11,7 @@ | |||
"@aws-sdk/client-athena": "^3.163.0", | |||
"@aws-sdk/client-s3": "^3.120.0", | |||
"@aws-sdk/lib-storage": "^3.121.0", | |||
"@iotexproject/iotex-address-ts": "^1.0.4", | |||
"iotex-antenna": "^0.31.3" | |||
"@iotexproject/iotex-address-ts": "^1.0.4" |
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.
rm
apps/web/package.json
Outdated
"util": "^0.12.5", | ||
"vite-plugin-pages": "^0.28.0", |
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.
rm
@shanejearley - Removed
iotex-antenna
npm package as well asiopay.ts
composable. Interfacing with IOTX using MetaMask includingswitchNetwork
functionality is currently enabled.@shanejearley & @hawyar - Please note that I did not remove
iotex-antenna
from theservices/crawler
/@casimir/crawler
package. Will defer to you guys there. If you'd like me to remove that in this PR, just let me know.Note: Found a bug with what appears to be missing polyfills for the(All lies. Removed that garbage.)crypto
package, which is a dependency within one of thenode_modules
. The fix can be found invite.config.ts
as well as installingcrypto-browserify
npm package.