Skip to content

Conversation

scottinet
Copy link
Contributor

7.4.1 (2020-10-28)

Enhancements

Others

  • [ #548 ] [doc] Add a disclaimer about Expo in getting started React Native (Njuelle)
  • [ #547 ] Stick realtime:unsubscribe behavior to documentation (Yoann-Abbes)

Leodau and others added 9 commits September 2, 2020 16:29
Correctly adds empty lines between each snippet and the actual file content.

Fixes kuzzleio/documentation#454
What does this PR do?
Stick behavior of realtime:unsubscribe to documentation

Other change
Fix cypress tests by specifying CYPRESS_RECORD_KEY_DOC in package.json and Travis settings.
Since by default it uses CYPRESS_RECORD_KEY, it could not work since it's already used in the AC repo
Export types, classes and interfaces declared in interfaces.ts
# Description

Fix #549 

Update dependencies, and make this SDK compatible with Typescript 4. Errors were all due to WebSocket properties shadowing parent classes getters.

# Other changes

* Upgrade all dependencies
* Upgrade WebPack 4=>5 + configuration upgrade
* Deactivate the `no-shadow` eslint rule, and replace it with the dedicated TS one (the vanilla JS "no-shadow" rule doesn't work w/ ts code)
* Relax eslint rules for snippets a little, to allow our most recent coding styles in documentation snippets
@codecov
Copy link

codecov bot commented Oct 28, 2020

Codecov Report

Merging #553 into master will decrease coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #553      +/-   ##
==========================================
- Coverage   88.40%   88.39%   -0.01%     
==========================================
  Files          32       32              
  Lines        1449     1448       -1     
  Branches      252      252              
==========================================
- Hits         1281     1280       -1     
  Misses        119      119              
  Partials       49       49              
Impacted Files Coverage Δ
src/controllers/Realtime.ts 90.62% <ø> (-0.15%) ⬇️
src/protocols/WebSocket.ts 90.76% <ø> (ø)
src/protocols/abstract/Base.ts 91.66% <ø> (ø)

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 4515cdf...a20cd6d. Read the comment docs.

@scottinet scottinet merged commit fc45968 into master Oct 28, 2020
@scottinet scottinet deleted the 7.4.1-proposal branch October 28, 2020 15:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants