Skip to content

Conversation

@BelfordZ
Copy link
Contributor

@BelfordZ BelfordZ commented Nov 7, 2023

Explanation

Release queued request controller with fixes for network nickname & addEthereumChain

References

Changelog

Checklist

  • I've updated the test suite for new or updated code as appropriate
  • I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
  • I've highlighted breaking changes using the "BREAKING" category above as appropriate

@BelfordZ BelfordZ requested a review from a team as a code owner November 7, 2023 20:33
adonesky1
adonesky1 previously approved these changes Nov 7, 2023
Copy link
Contributor

@adonesky1 adonesky1 left a comment

Choose a reason for hiding this comment

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

LGTM!

jiexi
jiexi previously approved these changes Nov 7, 2023
- Bump @metamask/utils from 8.1.0 to 8.2.0 ([#1957](https://github.com/MetaMask/core/pull/1957))

### Fixed
- Fix network nicknames ([#2000](https://github.com/MetaMask/core/pull/2000))
Copy link
Contributor

@mcmire mcmire Nov 7, 2023

Choose a reason for hiding this comment

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

What about nicknames got fixed? Is it that they're now available, and if so, available how? Also, it's not just nickname but theoretically any other network configuration information yes? It would be nice to be more clear here for other consumers.


### Fixed
- Fix network nicknames ([#2000](https://github.com/MetaMask/core/pull/2000))
- Fix addEthereumChain showing switch ethereum chain dialog ([#2000](https://github.com/MetaMask/core/pull/2000))
Copy link
Contributor

@mcmire mcmire Nov 7, 2023

Choose a reason for hiding this comment

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

Is that it did show it before and does not now, or the opposite? It would be nice to be more clear here for other consumers.

@BelfordZ BelfordZ requested a review from mcmire November 7, 2023 21:26
Copy link
Contributor

@mcmire mcmire left a comment

Choose a reason for hiding this comment

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

Looks good. Thanks!

@BelfordZ BelfordZ merged commit 7c74b62 into main Nov 7, 2023
@BelfordZ BelfordZ deleted the release/89.0.0 branch November 7, 2023 21:43
MajorLift added a commit that referenced this pull request Nov 7, 2023
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.

5 participants