Skip to content
This repository was archived by the owner on Sep 28, 2022. It is now read-only.

Conversation

@samniisan
Copy link
Contributor

@samniisan samniisan commented Jun 28, 2017

Introduces

  • Meta in Security documents (User, Profile, Role)

Boyscoot

  • Updated get Security document API endpoint (fetchUser|Profile|Role -> getUser|Profile|Role)

Should only be merged after Kuzzle side PR
kuzzleio/kuzzle#897

@codecov-io
Copy link

codecov-io commented Jun 28, 2017

Codecov Report

Merging #144 into 3.x will decrease coverage by 0.08%.
The diff coverage is 87.8%.

Impacted file tree graph

@@            Coverage Diff             @@
##              3.x     #144      +/-   ##
==========================================
- Coverage   91.57%   91.49%   -0.09%     
==========================================
  Files          32       32              
  Lines        4678     4689      +11     
  Branches      437      438       +1     
==========================================
+ Hits         4284     4290       +6     
- Misses        268      272       +4     
- Partials      126      127       +1
Impacted Files Coverage Δ
src/main/java/io/kuzzle/sdk/security/Profile.java 93.1% <100%> (ø) ⬆️
src/main/java/io/kuzzle/sdk/security/User.java 95.95% <100%> (ø) ⬆️
src/main/java/io/kuzzle/sdk/core/Kuzzle.java 92.24% <100%> (ø) ⬆️
src/main/java/io/kuzzle/sdk/security/Role.java 100% <100%> (ø) ⬆️
.../kuzzle/sdk/security/AbstractSecurityDocument.java 93.84% <77.77%> (-2.65%) ⬇️
src/main/java/io/kuzzle/sdk/security/Security.java 93.1% <89.28%> (-0.43%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ee647f9...aa60055. Read the comment docs.

@scottinet scottinet merged commit 42ed42d into 3.x Jul 3, 2017
@scottinet scottinet deleted the 36-security-meta branch July 3, 2017 09:50
@stafyniaksacha stafyniaksacha mentioned this pull request Jul 19, 2017
@scottinet scottinet mentioned this pull request Aug 28, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants