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

Kuz 393 security user create user id not read #144

Merged
merged 6 commits into from
Feb 1, 2016

Conversation

benoitvidis
Copy link
Contributor

⚠️ dependent on PR #141

Bugfix. The securityUser::createUser action would not hydrate the proper User object.

@codecov-io
Copy link

Current coverage is 91.82%

Merging #144 into master will not affect coverage as of 16adee0

@@            master    #144   diff @@
======================================
  Files           77      77       
  Stmts         3925    3929     +4
  Branches         0       0       
  Methods          0       0       
======================================
+ Hit           3604    3608     +4
  Partial          0       0       
  Missed         321     321       

Review entire Coverage Diff as of 16adee0


Uncovered Suggestions

  1. +0.26% via ...b/services/rabbit.js#295...304
  2. +0.23% via lib/api/enable.js#14...22
  3. +0.21% via lib/api/perf.js#80...87
  4. See 7 more...

Powered by Codecov. Updated on successful CI builds.

@scottinet
Copy link
Contributor

+1

1 similar comment
@j33f
Copy link

j33f commented Feb 1, 2016

+1

AnthonySendra added a commit that referenced this pull request Feb 1, 2016
…r-id-not-read

Kuz 393 security user create user id not read
@AnthonySendra AnthonySendra merged commit 38bc1d3 into master Feb 1, 2016
@AnthonySendra AnthonySendra deleted the kuz-393-security-user-createUser-id-not-read branch February 1, 2016 13:55
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.

5 participants