Skip to content

Conversation

farhanW3
Copy link
Contributor

Changes

  • Gas issue on sdk solved with sdk update & using retry env variable
  • Updated sdk & SQL use MAX_RETRIES_FOR_TX

How this PR will be tested

  • Use /write contract end-point on mumbai & see if the transaction goes through

Output

Worker should not give gas error & transaction is submitted.

@zeet-co
Copy link

zeet-co bot commented Sep 14, 2023

We're building your pull request over on Zeet.
Click me for more info about your build and deployment.
Once built, this branch can be tested at: https://web3-api-akbv-farhan-sdk-gas-upd.chainsaw-dev.zeet.app before merging 😉

@socket-security
Copy link

New dependencies detected. Learn more about Socket for GitHub ↗︎

Packages Version New capabilities Transitives Size Publisher
@thirdweb-dev/sdk 3.10.60 None +2 55.1 MB jnsdls

@farhanW3 farhanW3 merged commit 4d3b359 into main Sep 14, 2023
@farhanW3 farhanW3 deleted the farhan/sdk-gas-upd branch September 14, 2023 05:07
farhanW3 added a commit that referenced this pull request Sep 14, 2023
* fix for adding wallets dynamically for a new chain (#128)

* Updated sdk to 3.10.60 & used retry var (#129)
adam-maj added a commit that referenced this pull request Sep 14, 2023
* updated roles/permission doc (#115)

* swagger UI on Production gives a 404 (#116)

* fixed nodemon & retry to not update mined status, let the other process handle it (#117)

* Retry Fixes  (#123)

* updated retry to not retry if gas value is lower than previously sent. Dynamic Wallet Add doesn't work. Will be fixed in Adam/Furqans branch

* updated log to warn & updated message

* Testjam error fixes (#124)

* fixed Jorge worker error

* readme update for swagger UI for Docker Image

* added examples for end-points

* enabled swagger UI on production (#127)

* fix for adding wallets dynamically for a new chain (#128)

* Updated sdk to 3.10.60 & used retry var (#129)

* Fix prebuilt merge conflicts

* Remove dbOperations
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant