search

accounts-js / accounts

Fullstack authentication and accounts-management for Javascript.
https://www.accountsjs.com/
MIT License
1.5k stars 141 forks source link

fix: fix graphql api error with apollo subscriptions #1137

Closed pradel closed 3 years ago

pradel commented 3 years ago

Fixes #1102 and #843

changeset-bot[bot] commented 3 years ago

🦋 Changeset detected

Latest commit: d1ac82ee3d2e0725f02df4ae565db600c27365fc

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package | Name | Type | | --------------------- | ----- | | @accounts/graphql-api | Patch |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

codecov[bot] commented 3 years ago

Codecov Report

Merging #1137 (d1ac82e) into master (217004d) will decrease coverage by 0.04%. The diff coverage is 50.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1137      +/-   ##
==========================================
- Coverage   95.53%   95.49%   -0.05%     
==========================================
  Files          93       93              
  Lines        2150     2152       +2     
  Branches      401      402       +1     
==========================================
+ Hits         2054     2055       +1     
- Misses         94       95       +1     
  Partials        2        2
Impacted Files Coverage Δ
packages/graphql-api/src/modules/accounts/index.ts 78.26% <ø> (ø)
packages/graphql-api/src/utils/context-builder.ts 90.47% <50.00%> (-4.27%) :arrow_down:

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 217004d...d1ac82e. Read the comment docs.