Skip to content

Make resetConnection async - #1239

Merged
Gudahtt merged 2 commits into
mainfrom
make-reset-connection-async
Apr 21, 2023
Merged

Make resetConnection async#1239
Gudahtt merged 2 commits into
mainfrom
make-reset-connection-async

Conversation

@Gudahtt

Copy link
Copy Markdown
Member

Description

The network controller method resetConnection has been made async. It will not resolve until after the network has been fully reset.

Changes

  • CHANGED: resetConnection is now async

References

Relates to #1176

Checklist

  • I've updated the test suite for new or updated code as appropriate
  • I've updated documentation for new or updated code as appropriate (note: this will usually be JSDoc)
  • I've highlighted breaking changes using the "BREAKING" category above as appropriate

@Gudahtt
Gudahttforce-pushed the make-rollback-to-previous-provider-async branch from 0415411 to 948e8aeCompareApril 21, 2023 00:59
@Gudahtt
Gudahttforce-pushed the make-reset-connection-async branch from a047b9e to 95f7ea9CompareApril 21, 2023 00:59
@Gudahtt

This comment was marked as resolved.

@Gudahtt
Gudahttforce-pushed the make-rollback-to-previous-provider-async branch from 948e8ae to 7f401cdCompareApril 21, 2023 12:49
@Gudahtt
Gudahttforce-pushed the make-reset-connection-async branch from 95f7ea9 to c8fd207CompareApril 21, 2023 12:50
@Gudahtt
Gudahtt changed the base branch from make-rollback-to-previous-provider-async to update-eip-1559-compatibility-during-network-lookupApril 21, 2023 12:50
@Gudahtt
Gudahttforce-pushed the update-eip-1559-compatibility-during-network-lookup branch from eb4fae4 to 4e1f13cCompareApril 21, 2023 12:51
@Gudahtt
Gudahttforce-pushed the make-reset-connection-async branch from c8fd207 to 70f27b1CompareApril 21, 2023 12:51
Base automatically changed from update-eip-1559-compatibility-during-network-lookup to mainApril 21, 2023 13:27
The network controller method `resetConnection` has been made async. It
will not resolve until after the network has been fully reset.
Relates to #1176
@Gudahtt
Gudahttforce-pushed the make-reset-connection-async branch from 70f27b1 to 7c0f70fCompareApril 21, 2023 13:28
@Gudahtt
Gudahtt marked this pull request as ready for review April 21, 2023 13:28
@Gudahtt
Gudahtt requested a review from a team as a code ownerApril 21, 2023 13:28

@cryptodev-2scryptodev-2s left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@Gudahtt
Gudahtt merged commit 1137676 into mainApr 21, 2023
@Gudahtt
Gudahtt deleted the make-reset-connection-async branch April 21, 2023 14:22
@legobeatlegobeat mentioned this pull request Apr 25, 2023
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.

2 participants

@Gudahtt@cryptodev-2s