-
Notifications
You must be signed in to change notification settings - Fork 29
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/gas config #331
Feat/gas config #331
Conversation
Develop to main
Develop to main
Release v0.9.0
Release v0.9.1
Release v0.9.2
Release v0.10.0
Release v0.11.0
Release v0.11.1
Release v0.12.0
Release v0.13.0
Release v0.14.0-beta.0
Release v0.14.0
Release v0.14.1
Release v0.14.3
chore: release v0.14.4
chore: release v0.15.0
Release v0.16.0
Release v0.17.0
Release v0.17.1
…18.0 Release v0.18.0
Update main branch to sync latest docs
…ase--branches--main--components--erc725.js chore(main): release 0.19.0
…ase--branches--main--components--erc725.js chore(main): release 0.20.0
…ase--branches--main--components--erc725.js chore(main): release 0.20.1
Codecov Report
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. @@ Coverage Diff @@
## develop #331 +/- ##
===========================================
+ Coverage 83.04% 83.71% +0.67%
===========================================
Files 18 18
Lines 979 1130 +151
Branches 218 255 +37
===========================================
+ Hits 813 946 +133
- Misses 91 98 +7
- Partials 75 86 +11
|
@kalote you need to checkout the branch from The diffs are not showing properly as a result. |
@kalote @CJ42 The ipfsGateway thing will be new with the new IPFS library. I create a token authorized proxy which is compatible with that library but @CallumGrindle had problems using it in nodejs, so it may currently only work in the browser. I am working on it. |
What kind of change does this PR introduce (bug fix, feature, docs update, ...)?
New
gas
configuration in the optionalparams
setting:I also removed the
gasPrice
from the constants.ts file as it was unused.What is the current behaviour (you can also link to an open issue here)?
Fix #328
What is the new behaviour (if this is a feature change)?
Other information: