Skip to content

Conversation

benoitvidis
Copy link
Contributor

@benoitvidis benoitvidis commented Jul 9, 2019

What does this PR do?

Add documentation for the security controller.

⚠️ Once this PR is merged, the documentation submodule needs to be updated!

Other changes

  • add a npm run doc command to serve the documentation in development mode
  • remove unused (for now) dependency from travis file.

@codecov
Copy link

codecov bot commented Jul 9, 2019

Codecov Report

Merging #417 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master    #417   +/-   ##
======================================
  Coverage    96.3%   96.3%           
======================================
  Files          32      32           
  Lines        1517    1517           
======================================
  Hits         1461    1461           
  Misses         56      56

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 ec85a14...cd6882f. Read the comment docs.

@codecov
Copy link

codecov bot commented Jul 9, 2019

Codecov Report

Merging #417 into master will increase coverage by 0.1%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master     #417     +/-   ##
=========================================
+ Coverage    96.3%   96.41%   +0.1%     
=========================================
  Files          32       32             
  Lines        1517     1535     +18     
=========================================
+ Hits         1461     1480     +19     
+ Misses         56       55      -1
Impacted Files Coverage Δ
src/protocols/http.js 92.68% <100%> (+0.27%) ⬆️
src/controllers/bulk.js 100% <100%> (ø) ⬆️
src/core/Jwt.js 100% <100%> (+4.76%) ⬆️
src/protocols/abstract/common.js 91.11% <100%> (+2.22%) ⬆️

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 58d7837...a16558c. Read the comment docs.

Copy link
Contributor

@Aschen Aschen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Congrats !

@codecov-io
Copy link

Codecov Report

Merging #417 into master will increase coverage by 0.14%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #417      +/-   ##
==========================================
+ Coverage    96.3%   96.45%   +0.14%     
==========================================
  Files          32       33       +1     
  Lines        1517     1550      +33     
==========================================
+ Hits         1461     1495      +34     
+ Misses         56       55       -1
Impacted Files Coverage Δ
src/protocols/http.js 92.68% <100%> (+0.27%) ⬆️
src/controllers/bulk.js 100% <100%> (ø) ⬆️
src/core/Jwt.js 100% <100%> (+4.76%) ⬆️
src/protocols/abstract/pendingRequest.js 100% <100%> (ø)
src/protocols/abstract/common.js 91.83% <100%> (+2.94%) ⬆️
src/protocols/abstract/realtime.js 100% <100%> (ø) ⬆️
src/Kuzzle.js 94.59% <100%> (ø) ⬆️

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 58d7837...e24a2e5. Read the comment docs.

@codecov-io
Copy link

codecov-io commented Jul 22, 2019

Codecov Report

Merging #417 into 6-dev will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##            6-dev     #417   +/-   ##
=======================================
  Coverage   96.45%   96.45%           
=======================================
  Files          33       33           
  Lines        1550     1550           
=======================================
  Hits         1495     1495           
  Misses         55       55

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 83a3868...b9fa7ce. Read the comment docs.

@scottinet scottinet changed the base branch from master to 6-dev July 23, 2019 08:01
<br />

```js
createFirstAdmin(kuid, body, [options]);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The kuid is optional and should be put in the options object

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is not in the sdk and would induce a breaking change.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we should fix our documentation then

@scottinet
Copy link
Contributor

I took the liberty to change the target base from master to 6-dev.

benoitvidis and others added 11 commits July 24, 2019 11:47
…ate-or-replace-role.test.yml

Co-Authored-By: Sébastien Cottinet <[email protected]>
…ate-restricted-user.test.yml

Co-Authored-By: Sébastien Cottinet <[email protected]>
…credentials.test.yml

Co-Authored-By: Sébastien Cottinet <[email protected]>
@scottinet scottinet merged commit bace292 into 6-dev Jul 25, 2019
@scottinet scottinet deleted the kzl-1174-doc-security-controller branch July 25, 2019 09:42
@scottinet scottinet mentioned this pull request Jul 31, 2019
scottinet added a commit that referenced this pull request Jul 31, 2019
# [6.2.0](https://github.com/kuzzleio/sdk-javascript/releases/tag/6.2.0) (2019-07-31)


#### Bug fixes

- [ [#428](#428) ] Properly handle boolean flags in HTTP querystrings   ([scottinet](https://github.com/scottinet))
- [ [#427](#427) ] Solve promise+event+memory leaks when the network fails   ([scottinet](https://github.com/scottinet))
- [ [#424](#424) ] Prevent pending request leak when disconnect the SDK   ([Aschen](https://github.com/Aschen))
- [ [#422](#422) ] Fix bug when decoding JWT in browser   ([Aschen](https://github.com/Aschen))
- [ [#420](#420) ] Fix http protocol unresolved promise on connection error   ([Aschen](https://github.com/Aschen))

#### New features

- [ [#419](#419) ] Add bulk:write and bulk:mWrite   ([Aschen](https://github.com/Aschen))

#### Enhancements

- [ [#421](#421) ] Get api routes from server:publicApi   ([Aschen](https://github.com/Aschen))
- [ [#423](#423) ] Emit queryError event on malformed request   ([Aschen](https://github.com/Aschen))
- [ [#417](#417) ] Security controller documentation   ([benoitvidis](https://github.com/benoitvidis))
---
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants