Skip to content

Conversation

benoitvidis
Copy link
Contributor

This PR allows the sdk to get the additional errors and count properties when receiving a PartialError from Kuzzle.

@codecov-io
Copy link

codecov-io commented Feb 25, 2019

Codecov Report

Merging #379 into 6-dev will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##            6-dev     #379      +/-   ##
==========================================
+ Coverage   96.71%   96.72%   +<.01%     
==========================================
  Files          30       30              
  Lines        1431     1434       +3     
==========================================
+ Hits         1384     1387       +3     
  Misses         47       47
Impacted Files Coverage Δ
src/KuzzleError.js 100% <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 3876979...1771188. Read the comment docs.

@jenow jenow merged commit a18be01 into 6-dev Feb 25, 2019
@jenow jenow deleted the bugfix/partialerrors-missing-internal-errors branch February 25, 2019 14:07
@scottinet scottinet mentioned this pull request Mar 26, 2019
scottinet added a commit that referenced this pull request Mar 27, 2019
# [6.1.0](https://github.com/kuzzleio/sdk-javascript/releases/tag/6.1.0) (2019-03-26)


#### New features

- [ [#381](#381) ] Add support for the new auth:refreshToken API route   ([scottinet](https://github.com/scottinet))

#### Others

- [ [#379](#379) ] [bugfix] keep internal errors on PartialErrors   ([benoitvidis](https://github.com/benoitvidis))
- [ [#376](#376) ] [ci] Add empty .npmignore to push dist on npmjs   ([alexandrebouthinon](https://github.com/alexandrebouthinon))
---
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