fix(arweave): modify import of Arweave in ArweaveToken
#200
Conversation
Temporary patch for issues related to using `.init()` in web environments
| @@ -54,7 +54,7 @@ export class ArweaveToken implements TokenTools { | |||
|
|
|||
| this.arweave = | |||
| arweave ?? | |||
| Arweave.init({ | |||
There was a problem hiding this comment.
@fedellen - the only SDK change, i think it's just how bundlers tree shake the arweave import here and they use node by default - causing the default.init() not to exist in web projects. more to understand there but worked with my local example so will ask others to test.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## alpha #200 +/- ##
=======================================
Coverage 93.60% 93.60%
=======================================
Files 25 25
Lines 3485 3486 +1
Branches 159 159
=======================================
+ Hits 3262 3263 +1
Misses 223 223 ☔ View full report in Codecov by Sentry. |
|
🎉 This PR is included in version 1.20.0-alpha.3 🎉 The release is available on: Your semantic-release bot 📦🚀 |
|
🎉 This PR is included in version 1.20.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Temporary patch for issues related to using
.init()in web environments