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.

Sign up for GitHub

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

Jump to bottom

v11.1.0 release #514

Closed
wants to merge 1 commit into from
Closed

v11.1.0 release #514

wants to merge 1 commit into from

Conversation

brad-decker
Copy link
Contributor

@brad-decker brad-decker commented Jul 7, 2021 •
edited
Loading

Proposed Changelog:

[11.1.0] - 2021-07-07

Added

  • Added GasFeeController to provide gas fee estimates #494

Changed

  • Added chainId support to TokenRatesControllers #476
  • Added iconUrl to Token type in TokenListController #512

@brad-decker brad-decker requested a review from a team as a code owner July 7, 2021 18:48
@wachunei
Copy link
Member

wachunei commented Jul 7, 2021

I think this should be 12, because of this: #476 (comment), but mobile has not adopted 11 yet so we could make an exception perhaps? :unsure:

@brad-decker
Copy link
Contributor Author

I'm fine making it v12 @Gudahtt do you have opinions?

@Gudahtt
Copy link
Member

Gudahtt commented Jul 7, 2021

I agree with @wachunei ; this should be v12 because of the linked breaking change. Lets make the update as easy as possible by highlighting the required changes.

@brad-decker
Copy link
Contributor Author

Closing in favor of #515

@brad-decker brad-decker closed this Jul 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@brad-decker @wachunei @Gudahtt