Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

address book entries by chainId #7205

Merged
merged 5 commits into from
Sep 21, 2019
Merged

address book entries by chainId #7205

merged 5 commits into from
Sep 21, 2019

Conversation

jennypollack
Copy link
Contributor

@jennypollack jennypollack commented Sep 20, 2019

Updates extension to reflect change in AddressBookController: MetaMask/core#152

@metamaskbot
Copy link
Collaborator

Builds ready [9ed8f82]

@metamaskbot
Copy link
Collaborator

Builds ready [4debc78]

Gudahtt
Gudahtt previously approved these changes Sep 20, 2019
Copy link
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.

Looks good to me!

@metamaskbot
Copy link
Collaborator

Builds ready [029b0de]

@metamaskbot
Copy link
Collaborator

Builds ready [70942e9]

Copy link
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.

Whoops, I merged something else that conflicted with this lockfile. I've just rebased it to resolve the conflict.

@metamaskbot
Copy link
Collaborator

Builds ready [0e46617]

@Gudahtt Gudahtt merged commit e86cebd into develop Sep 21, 2019
Gudahtt added a commit to Gudahtt/metamask-extension that referenced this pull request Sep 27, 2019
* origin/develop: (56 commits)
  Add advanced setting to enable editing nonce on confirmation screens (MetaMask#7089)
  Add migration on 3box imports and remove feature flag (MetaMask#7209)
  ci - install deps - limit install scripts to whitelist (MetaMask#7208)
  Add a/b test for full screen transaction confirmations (MetaMask#7162)
  Update minimum Firefox verison to 56.0 (MetaMask#7213)
  mesh-testing - submit infura rpc requests to mesh-testing container (MetaMask#7031)
  obs-store/local-store should upgrade webextension error to real error (MetaMask#7207)
  sesify-viz - bump dep for visualization enhancement (MetaMask#7175)
  address book entries by chainId (MetaMask#7205)
  Optimize images only during production build (MetaMask#7194)
  Use common test build during CI (MetaMask#7196)
  Report missing `en` locale messages to Sentry (MetaMask#7197)
  Verify locales on CI (MetaMask#7199)
  updated ganache and addons-linter (MetaMask#7204)
  fixup! add user rejected errors
  add user rejected errors
  update json-rpc-engine
  use eth-json-rpc-errors
  Remove unused locale messages (MetaMask#7190)
  Remove unused components (MetaMask#7191)
  ...
@whymarrh whymarrh deleted the address-book-chainId branch October 2, 2019 15:16
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants