Skip to content
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

Implement GAS_LIMIT usage for python sdk #963

Merged
merged 1 commit into from
Oct 2, 2023

Conversation

alidzm
Copy link
Collaborator

@alidzm alidzm commented Oct 2, 2023

Description

To increase percentage of successfully handled transactions we need to provide custom gas limit to it to avoid out of gas errors.

Summary of changes

Add gas limit to transaction.

How test the changes

Define gas limit and deploy a job. Check gas limit of handled transaction.

Related issues

Keywords for linking issues

Operational checklist

  • All new functionality is covered by tests
  • Any related documentation has been changed or added

@vercel
Copy link

vercel bot commented Oct 2, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
human-protocol-typescript-sdk-docs ❌ Failed (Inspect) Oct 2, 2023 2:51pm
10 Ignored Deployments
Name Status Preview Comments Updated (UTC)
escrow-dashboard ⬜️ Ignored (Inspect) Visit Preview Oct 2, 2023 2:51pm
faucet-server ⬜️ Ignored (Inspect) Visit Preview Oct 2, 2023 2:51pm
fortune-exchange-oracle ⬜️ Ignored (Inspect) Visit Preview Oct 2, 2023 2:51pm
fortune-exchange-oracle-server ⬜️ Ignored (Inspect) Visit Preview Oct 2, 2023 2:51pm
fortune-job-launcher-client ⬜️ Ignored (Inspect) Visit Preview Oct 2, 2023 2:51pm
fortune-job-launcher-server ⬜️ Ignored (Inspect) Visit Preview Oct 2, 2023 2:51pm
fortune-recording-oracle ⬜️ Ignored (Inspect) Visit Preview Oct 2, 2023 2:51pm
fortune-reputation-oracle ⬜️ Ignored (Inspect) Visit Preview Oct 2, 2023 2:51pm
job-launcher-client ⬜️ Ignored (Inspect) Visit Preview Oct 2, 2023 2:51pm
job-launcher-server ⬜️ Ignored (Inspect) Visit Preview Oct 2, 2023 2:51pm

@alidzm alidzm changed the base branch from main to develop October 2, 2023 13:09
@vercel vercel bot temporarily deployed to Preview – human-protocol-typescript-sdk-docs October 2, 2023 13:10 Inactive
@alidzm alidzm force-pushed the fix/use_gas_limit_python_sdk branch from eb99351 to cecefb5 Compare October 2, 2023 14:04
@vercel vercel bot temporarily deployed to Preview – human-protocol-typescript-sdk-docs October 2, 2023 14:18 Inactive
@alidzm alidzm force-pushed the fix/use_gas_limit_python_sdk branch from cecefb5 to 9104580 Compare October 2, 2023 14:29
@vercel vercel bot temporarily deployed to Preview – human-protocol-typescript-sdk-docs October 2, 2023 14:34 Inactive
@alidzm alidzm force-pushed the fix/use_gas_limit_python_sdk branch from 9104580 to 4eb23ad Compare October 2, 2023 14:47
@vercel vercel bot temporarily deployed to Preview – human-protocol-typescript-sdk-docs October 2, 2023 14:51 Inactive
@leric7 leric7 merged commit c8b3713 into develop Oct 2, 2023
20 of 23 checks passed
@leric7 leric7 deleted the fix/use_gas_limit_python_sdk branch October 2, 2023 15:46
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.

2 participants