Skip to content

Conversation

Aschen
Copy link
Contributor

@Aschen Aschen commented Jul 22, 2020

What does this PR do?

Convert protocols classes to TS

Other changes

  • sslConnection is deprecated for ssl and the value is true by default for port 443 and 7443
  • remove the error.internal error who contain the original WS connect error and put the info in the message instead

@codecov
Copy link

codecov bot commented Jul 22, 2020

Codecov Report

Merging #529 into 7-dev will decrease coverage by 1.49%.
The diff coverage is 95.16%.

Impacted file tree graph

@@            Coverage Diff             @@
##            7-dev     #529      +/-   ##
==========================================
- Coverage   91.18%   89.68%   -1.50%     
==========================================
  Files          32       32              
  Lines        1395     1406      +11     
  Branches      133      225      +92     
==========================================
- Hits         1272     1261      -11     
- Misses         98       99       +1     
- Partials       25       46      +21     
Impacted Files Coverage Δ
src/Kuzzle.ts 87.50% <ø> (-0.06%) ⬇️
src/controllers/Auth.ts 75.64% <ø> (-0.31%) ⬇️
src/controllers/Base.ts 87.50% <ø> (-1.39%) ⬇️
src/controllers/Document.ts 70.78% <ø> (-0.33%) ⬇️
src/core/searchResult/SearchResultBase.ts 77.19% <ø> (-0.40%) ⬇️
src/protocols/abstract/Realtime.ts 97.14% <83.33%> (ø)
src/protocols/abstract/Base.ts 91.52% <86.66%> (ø)
src/KuzzleError.ts 100.00% <100.00%> (ø)
src/core/KuzzleEventEmitter.ts 90.00% <100.00%> (ø)
src/protocols/Http.ts 85.09% <100.00%> (ø)
... and 9 more

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 25fa21c...a2b2f2e. Read the comment docs.

Aschen and others added 4 commits August 6, 2020 10:30
Co-authored-by: Sébastien Cottinet <scottinet@protonmail.com>
Co-authored-by: Sébastien Cottinet <scottinet@protonmail.com>
…o/sdk-javascript into add-typescript-support-protocols
Aschen added 2 commits August 11, 2020 11:40
…o/sdk-javascript into add-typescript-support-protocols
@Aschen Aschen merged commit a167d04 into 7-dev Aug 12, 2020
@Aschen Aschen deleted the add-typescript-support-protocols branch August 12, 2020 07:44
This was referenced Aug 18, 2020
Aschen added a commit that referenced this pull request Aug 18, 2020
# [7.4.0](https://github.com/kuzzleio/sdk-javascript/releases/tag/7.4.0) (2020-08-18)


#### New features

- [ [#542](#542) ] Finalize typescript support   ([Aschen](https://github.com/Aschen))
- [ [#529](#529) ] Add typescript support for protocols   ([Aschen](https://github.com/Aschen))

#### Enhancements

- [ [#541](#541) ] Add meaningful stacktrace   ([Aschen](https://github.com/Aschen))
- [ [#537](#537) ] Add typescript support realtime   ([Aschen](https://github.com/Aschen))
- [ [#531](#531) ] Add typescript support for Index and Collection controllers   ([Aschen](https://github.com/Aschen))
- [ [#530](#530) ] Add typescript support for search results   ([Aschen](https://github.com/Aschen))
- [ [#523](#523) ] Add support for the collection:delete API action   ([morgandruesne](https://github.com/morgandruesne))
---
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.

3 participants