Skip to content
This repository has been archived by the owner on Nov 9, 2023. It is now read-only.

Add DuplexJsonRpcEngine #107

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from
Draft

Add DuplexJsonRpcEngine #107

wants to merge 7 commits into from

Conversation

rekmarks
Copy link
Member

@rekmarks rekmarks commented May 18, 2022

WIP.

Requires #104

@rekmarks rekmarks changed the base branch from main to handle-notifications May 18, 2022 07:57
Base automatically changed from handle-notifications to main August 15, 2022 22:59
@legobeat
Copy link
Contributor

legobeat commented Apr 19, 2023

@rekmarks Went ahead and rebased (resolving some minor conflicts) and addressing the lint issues. Now that depending changes have landed, do you think it's worthwhile to revisit this effort?

If so, what's outstanding?

@MajorLift
Copy link
Contributor

This library has now been migrated into the core monorepo. This PR has been locked and this repo will be archived shortly. Going forward, releases of this library will only include changes made in the core repo.

  • Please push this branch to core and open a new PR there.
  • Optionally, add a link pointing to the discussion in this PR to provide context.

@MetaMask MetaMask locked and limited conversation to collaborators Nov 7, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants