Skip to content

chore(request-node): eip-1559 for TheGraphDataAccess - #760

Merged
alexandre-abrioux merged 4 commits into
masterfrom
thegraph-data-access-gaslimit
Feb 7, 2022
Merged

chore(request-node): eip-1559 for TheGraphDataAccess#760
alexandre-abrioux merged 4 commits into
masterfrom
thegraph-data-access-gaslimit

Conversation

@alexandre-abrioux

@alexandre-abriouxalexandre-abrioux commented Feb 4, 2022

Copy link
Copy Markdown
Contributor

Description of the changes

Gas limit can be too low for transactions submitted through TheGraphDataAccess. This adds an EIP-1559 gas price estimation for transactions submitted with this data access implementation.

@alexandre-abriouxalexandre-abrioux changed the title chore(request-node): eip-1559 for storagechore(request-node): eip-1559 for TheGraphDataAccessFeb 4, 2022
@coveralls

coveralls commented Feb 4, 2022

Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 89.449% when pulling ed5892a on thegraph-data-access-gaslimit into 8d81be3 on master.

Comment threadpackages/request-node/src/thegraph/TheGraphStorage.ts Outdated
alexandre-abrioux added a commit that referenced this pull request Feb 5, 2022
@alexandre-abrioux
alexandre-abriouxforce-pushed the thegraph-data-access-gaslimit branch 2 times, most recently from 39154aa to a5bd64fCompareFebruary 5, 2022 14:11
Comment threadpackages/request-node/src/thegraph/TheGraphStorage.ts
Comment threadpackages/request-node/src/thegraph/TheGraphStorage.ts Outdated
Sign up for freeto 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.

6 participants

@alexandre-abrioux@coveralls@bertux@KolevDarko@benjlevesque@kevindavee