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.

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

Fix error when unset user email #1778

Merged
merged 1 commit into from
May 13, 2016

Conversation

Marco129
Copy link
Contributor

Fix issue #1768

@drew-gross
Copy link
Contributor

Can you update the test to try to read the email after the unset request returns and make sure it is actually gone?

@ghost
Copy link

ghost commented May 13, 2016

@Marco129 updated the pull request.

@codecov-io
Copy link

Current coverage is 92.24%

Merging #1778 into master will decrease coverage by -<.01%

  1. File src/rest.js (not in diff) was modified. more
    • Misses +1
    • Partials 0
    • Hits -1
@@             master      #1778   diff @@
==========================================
  Files            87         87          
  Lines          6201       6201          
  Methods        1070       1070          
  Messages          0          0          
  Branches       1289       1289          
==========================================
- Hits           5721       5720     -1   
- Misses          480        481     +1   
  Partials          0          0          

Powered by Codecov. Last updated by 19e7407...f8159fe

@drew-gross
Copy link
Contributor

Perfect

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.

3 participants