Skip to content

Conversation

@MichalLytek
Copy link
Owner

This PR adds a feature that automatically emit in schema all the object types that implements the interface types referenced in schema definition.

It also adds an option to disable the auto register feature per interface type, like the Node interface case for Relay-based system, in case of multiple schemas isolation.

Closes #568

@MichalLytek MichalLytek added the Enhancement 🆕 New feature or request label Apr 4, 2020
@MichalLytek MichalLytek added this to the 1.0.0 release milestone Apr 4, 2020
@MichalLytek MichalLytek self-assigned this Apr 4, 2020
@MichalLytek MichalLytek force-pushed the interface-auto-registering branch from 862ee22 to 331fe54 Compare April 4, 2020 17:05
@codecov
Copy link

codecov bot commented Apr 4, 2020

Codecov Report

Merging #595 into master will decrease coverage by 0.01%.
The diff coverage is 93.75%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #595      +/-   ##
==========================================
- Coverage   94.96%   94.94%   -0.02%     
==========================================
  Files          78       78              
  Lines        1390     1405      +15     
  Branches      271      275       +4     
==========================================
+ Hits         1320     1334      +14     
- Misses         67       68       +1     
  Partials        3        3              
Impacted Files Coverage Δ
src/decorators/InterfaceType.ts 100.00% <ø> (ø)
src/schema/schema-generator.ts 96.76% <93.75%> (-0.20%) ⬇️

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 9952196...6dcceac. Read the comment docs.

@MichalLytek MichalLytek mentioned this pull request Apr 4, 2020
@MichalLytek MichalLytek force-pushed the interface-auto-registering branch from 4bfe6fa to c759ff9 Compare April 4, 2020 17:26
@MichalLytek MichalLytek force-pushed the interface-auto-registering branch from c759ff9 to 7cbae98 Compare April 4, 2020 17:31
@MichalLytek MichalLytek force-pushed the interface-auto-registering branch from 7cbae98 to 6dcceac Compare April 4, 2020 17:34
@twavv
Copy link
Contributor

twavv commented Apr 5, 2020

Thanks for this, it looks good to me. :^)

@MichalLytek MichalLytek merged commit 41142d9 into master Apr 5, 2020
@MichalLytek MichalLytek deleted the interface-auto-registering branch April 5, 2020 11:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Enhancement 🆕 New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Orphaned Types

3 participants