Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix(stitching): avoid duplicate directives #1665

Merged
merged 1 commit into from Jun 21, 2020
Merged

fix(stitching): avoid duplicate directives #1665

merged 1 commit into from Jun 21, 2020

Commits on Jun 21, 2020

  1. fix(stitching): avoid duplicate directives 

    Fixes #1656.

Later directives with the same name should override earlier directives rather than causing an error.

Directives with the same names from a later subschema will override an earlier subschema. Directives from typedefs will override both.

Mixing of legacy schemas argument and newer subschemas, typeDefs, and types arguments should respect this same order.

We can consider adding onDirectiveConflict option to match onTypeConflict for customization.
    @yaacovCR
    yaacovCR committed Jun 21, 2020
    Copy the full SHA
    a279c84 View commit details
    Browse the repository at this point in the history