Skip to content

Fix connection removal bug#9137

Merged
rekmarks merged 2 commits intodevelopfrom
fix-removeConnection-bug
Aug 4, 2020
Merged

Fix connection removal bug#9137
rekmarks merged 2 commits intodevelopfrom
fix-removeConnection-bug

Conversation

@rekmarks
Copy link
Copy Markdown
Member

@rekmarks rekmarks commented Aug 4, 2020

Fixes what is effectively a type error that caused all connections for a given origin to be removed whenever one connection was removed.

Fixes #9126

@rekmarks rekmarks requested a review from a team as a code owner August 4, 2020 19:24
brad-decker
brad-decker previously approved these changes Aug 4, 2020
Copy link
Copy Markdown
Member

@Gudahtt Gudahtt left a comment

Choose a reason for hiding this comment

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

LGTM!

@rekmarks rekmarks merged commit 658e478 into develop Aug 4, 2020
@rekmarks rekmarks deleted the fix-removeConnection-bug branch August 4, 2020 20:02
@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [0779825]
Page Load Metrics (630 ± 36 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint297639105
domContentLoaded3457146287536
load3477156307636
domInteractive3457146287536

Gudahtt added a commit that referenced this pull request Aug 5, 2020
* origin/develop:
  Fix connection removal bug (#9137)
  Add source map validator to CI (#9135)
  Update source map validator target files (#9133)
  Improve sourcemap validator console report (#9131)
  Add `validate-source-maps` npm script (#9134)
  Non-zero exit code upon failure to validate source maps (#9132)
  remove unused tx-list styles (#9121)
  delete unused confirm styles (#9118)
Gudahtt pushed a commit that referenced this pull request Aug 7, 2020
* fix remove connections bug
@metamaskbot metamaskbot mentioned this pull request Aug 7, 2020
Gudahtt added a commit that referenced this pull request Aug 10, 2020
* origin/master: (44 commits)
  Add category in eventOpts (#9164)
  Update changelog for v8.0.7 (#9161)
  Version v8.0.7
  Remove web3 e2e tests (#9159)
  Add web3 usage metrics, prepare for web3 removal (#9144)
  Use `pathname` instead of URL for `currentPath` metrics parameter (#9158)
  Remove `url` parameter from `metricsEvent` (#9157)
  Change MetaMetrics category for background events (#9155)
  remove .network-name height
  Use luxon@1.24.1 (#9154)
  Update 'react-devtools' to ^4.8.0 (#9140)
  Fix connection removal bug (#9137)
  Add source map validator to CI (#9135)
  Update source map validator target files (#9133)
  Improve sourcemap validator console report (#9131)
  Add `validate-source-maps` npm script (#9134)
  Non-zero exit code upon failure to validate source maps (#9132)
  Update `brfs` from v1.6.1 to v2.0.2 (#9115)
  Factor out `getEnvironment` function in build script (#9114)
  Update `browserify` from v16.2.3 to v16.5.1 (#9113)
  ...
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.

MetaMesk does not always send the 'accountsChanged' event on v8.0.5

4 participants