Skip to content

enforce polygon maxPriorityFee minimum value as 30#1329

Merged
Jamchello merged 6 commits intodevelopfrom
polygon-gas-minimum
Jan 12, 2023
Merged

enforce polygon maxPriorityFee minimum value as 30#1329
Jamchello merged 6 commits intodevelopfrom
polygon-gas-minimum

Conversation

@Jamchello
Copy link
Copy Markdown
Contributor

fee transformer & tests added for polygon

@Jamchello Jamchello requested a review from mholtzman January 10, 2023 15:38
@Jamchello Jamchello changed the title enforce polygon maxBaseFee minimum value as 30 enforce polygon maxPriorityFee minimum value as 30 Jan 10, 2023
@goosewobbler goosewobbler added WIP PRs that are still in progress and not ready for review or merging gas labels Jan 10, 2023
* create per-chain gas calculators

* make it green

Co-authored-by: Jamie M <jamie.michel@hotmail.com>
@Jamchello Jamchello requested a review from mholtzman January 11, 2023 18:00
Copy link
Copy Markdown
Collaborator

@mholtzman mholtzman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR lgtm, just a couple of extreme nitpicks

@Jamchello Jamchello requested a review from mholtzman January 11, 2023 18:43
@Jamchello Jamchello merged commit 482c64e into develop Jan 12, 2023
@mholtzman mholtzman deleted the polygon-gas-minimum branch August 10, 2023 15:56
wakamex pushed a commit to wakamex/framed that referenced this pull request Mar 2, 2026
* fee transformer & tests added for polygon

* removed some styling changes

* Create gas calculators (floating#1331)

* create per-chain gas calculators

* make it green

Co-authored-by: Jamie M <jamie.michel@hotmail.com>

* condense feeHistory

* use resolves.toBe api

* update tests

Co-authored-by: Matt Holtzman <matt.holtzman@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gas WIP PRs that are still in progress and not ready for review or merging

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants