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

fix issue where ethQuery is not instantiated correctly when GasFeeController is constructed #936

Merged
merged 3 commits into from
Oct 13, 2022

Conversation

adonesky1
Copy link
Contributor

@adonesky1 adonesky1 commented Oct 12, 2022

  • FIXED:

    • Fixes an issue (I) introduced in this commit in v32.0.1 where an instance of ethQuery is not instantiated correctly when GasFeeController is constructed, which is currently causing an E2E test failure on this PR in the extension.

@adonesky1 adonesky1 requested a review from a team as a code owner October 12, 2022 23:56
@adonesky1 adonesky1 requested review from BelfordZ and Gudahtt October 12, 2022 23:59
@adonesky1 adonesky1 requested a review from Gudahtt October 13, 2022 14:14
Gudahtt
Gudahtt previously approved these changes Oct 13, 2022
Copy link
Member

@Gudahtt Gudahtt left a comment

Choose a reason for hiding this comment

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

LGTM! Good find, not sure how I missed this.

@adonesky1 adonesky1 merged commit aaa85c0 into main Oct 13, 2022
@adonesky1 adonesky1 deleted the fix-ethquery-gasFeeController branch October 13, 2022 16:01
@adonesky1 adonesky1 mentioned this pull request Oct 13, 2022
gantunesr pushed a commit that referenced this pull request Dec 8, 2022
…troller is constructed (#936)

* fix issue where ethQuery is not instantiated correctly when GasFeeController is constructed
MajorLift pushed a commit that referenced this pull request Oct 11, 2023

Verified

This commit was signed with the committer’s verified signature.
MajorLift Jongsun Suh
…troller is constructed (#936)

* fix issue where ethQuery is not instantiated correctly when GasFeeController is constructed
MajorLift pushed a commit that referenced this pull request Oct 11, 2023

Verified

This commit was signed with the committer’s verified signature.
MajorLift Jongsun Suh
…troller is constructed (#936)

* fix issue where ethQuery is not instantiated correctly when GasFeeController is constructed
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.

None yet

2 participants