Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Fix metamask notification #11539

Closed
wants to merge 1 commit into from
Closed

Fix metamask notification #11539

wants to merge 1 commit into from

Conversation

darkdh
Copy link
Member

@darkdh darkdh commented Oct 15, 2017

fix #11535
fix #11536

Auditors: @bbondy, @diracdeltas

Test Plan:

Submitter Checklist:

  • Submitted a ticket for my issue if one did not already exist.
  • Used Github auto-closing keywords in the commit message.
  • Added/updated tests for this change (for new code or code which already has tests).
  • Ran git rebase -i to squash commits (if needed).
  • Tagged reviewers and labelled the pull request as needed.

Test Plan:

Reviewer Checklist:

Tests

  • Adequate test coverage exists to prevent regressions
  • Tests should be independent and work correctly when run individually or as a suite ref
  • New files have MPL2 license header

@darkdh darkdh self-assigned this Oct 15, 2017
@darkdh darkdh requested review from diracdeltas and bbondy October 15, 2017 04:30
@bbondy bbondy closed this Oct 15, 2017
@darkdh
Copy link
Member Author

darkdh commented Oct 15, 2017

cherry-picked to 24b6cb9

@bsclifton bsclifton deleted the metamask-notification branch October 16, 2017 05:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants