Version v7.7.9 RC #8444
Merged
Version v7.7.9 RC #8444
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Builds ready [a7911ce]
|
This is a backport of #8314. Here's the original description: MetaMask would sometimes get into a state where the notification popup would never open. This could happen if the notification window was closed shortly after being opened. After this happened, no popups would show up until after the extension was reset. This was happening because the background thought the popup was already open. The variable it uses to track whether the popup was open or not was being set to `true` immediately after the background asked the browser to open a new window, before a handler was attached that could respond to the window being closed. Removing this line seems to solve the problem. This line was added originally in #5437, which dealt with batch transactions. Batches of transactions seem to work just fine without this line though (from local testing), and I can't think of why this would be required. Closes #7051
Builds ready [2a51399]
|
Builds ready [8fb615c]
|
This backports the deletion of the Dai/Sai migration notification (#8418). Note that the migration to delete the now unused background state has not been included, as it is non-essential and would have been more difficult to backport. The migration to delete the unused state will be included in the next major release instead.
Builds ready [57fdc03]
|
Backport #7654 to v7.7.9 Co-authored-by: Tyson Malchow <[email protected]>
This is backported to make it easier to backport #7986 Co-authored-by: Whymarrh Whitby <[email protected]>
Builds ready [50b48b7]
|
Backport #8363 to v7.7.9. Note that this uses `clone` instead of `cloneDeep`, because `clone` hadn't yet been replaced by `cloneDeep` on `master`. Backporting that change as well would have been very disruptive, so I've updated this to use `clone` instead to minimize conflicts. It is functionally equivalent. Co-authored-by: Whymarrh Whitby <[email protected]>
Builds ready [f91bd3a]
|
Backport #8200 to v7.7.9. Original commit description: The method registry was being initialized with the global variable `ethereumProvider` before that variable was set. As a result, the method registry was falling back to an internally constructed provider that used the wrong provider URL (an obsolete Infura API). This was resulting in an error with the message "Project ID not found". The method registry is now initialized lazily, when it's first needed. This should be well after the initialization of `ethereumProvider`, which occurs during the UI initialization.
Builds ready [3dc8387]
|
) Backport #7986 onto v7.7.9. The original commit message follows: With the change from infura to cryptocompare https://github.com/MetaMask/gaba/pull/30/files#diff-50c3c47cc5fa12e5213a6cc900476f41L41-R48 we have numerous conversion rates to go through and add if we like to. Co-authored-by: Thomas Huang <[email protected]>
Builds ready [361c571]
|
Backport #8197 to v7.7.9. Co-authored-by: Brad Decker <[email protected]>
Backport #7673 to v7.7.9. Co-authored-by: Whymarrh Whitby <[email protected]>
Backport #7762 to v7.7.9 Co-authored-by: Sirius Tsou <[email protected]>
…8465) Backport #7960 to v7.7.9 Co-authored-by: Whymarrh Whitby <[email protected]>
Backport #7464 to v7.7.9 Co-authored-by: Alice Henshaw <[email protected]> Co-authored-by: Whymarrh Whitby <[email protected]>
Builds ready [2d36d42]
|
Backport #8466 to v7.7.9 Co-authored-by: Erik Marks <[email protected]>
Builds ready [8d8a17d]
|
This was referenced May 4, 2020
Closed
tmashuang
previously approved these changes
May 4, 2020
Backport #7808 to v7.7.9 Co-authored-by: Lenard Frommelt <[email protected]>
tmashuang
previously approved these changes
May 4, 2020
Builds ready [b81558a]
|
tmashuang
dismissed
their stale review
Dissmissing review to check for trezor connectivity.
tmashuang
approved these changes
May 4, 2020
Gudahtt
approved these changes
May 4, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📦 🚀
Changelog: