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

Bring json-rpc-engine types up to date #19

Closed
wants to merge 2 commits into from

Conversation

mcmire
Copy link
Contributor

@mcmire mcmire commented Oct 5, 2022

A while back, a selection of types from json-rpc-engine were copied to this repo. We'd like to remove those types from json-rpc-engine as a result, however, since then, the types in json-rpc-engine have received updates that are not reflected here. This commit copies the new changes over.

A while back, a selection of types from `json-rpc-engine` were copied to
this repo. We'd like to remove those types from `json-rpc-engine` as a
result, however, since then, the types in `json-rpc-engine` have
received updates that are not reflected here. This commit copies the new
changes over.
@mcmire mcmire requested a review from a team as a code owner October 5, 2022 19:31
@mcmire
Copy link
Contributor Author

mcmire commented Oct 5, 2022

@Mrtenz
Copy link
Member

Mrtenz commented Mar 31, 2023

Could these be moved to @metamask/utils instead?

@mcmire
Copy link
Contributor Author

mcmire commented Apr 14, 2023

Yup, I'll work on a way to do that. Closing this.

@mcmire mcmire closed this Apr 14, 2023
@Mrtenz
Copy link
Member

Mrtenz commented Apr 14, 2023

Great. I've already moved some types to @metamask/permission-controller: MetaMask/core#1184.

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.

2 participants