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.

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

fix: include full @link directive definition #352

Merged
merged 1 commit into from
Sep 14, 2023

Conversation

dariuszkuc
Copy link
Member

Add missing Purpose enum. Full @link definition as per the spec:

directive @link(
    url: String!,
    as: String,
    import: [Import],
    for: Purpose)
repeatable on SCHEMA

scalar Import

enum Purpose {
  SECURITY
  EXECUTION
}

Resolves: #351

Add missing `Purpose` enum. Full `@link` definition as per the [spec](https://github.com/apollographql/specs/blob/main/link/v1.0/link-v1.0.graphql):

```graphql
directive @link(
    url: String!,
    as: String,
    import: [Import],
    for: Purpose)
repeatable on SCHEMA

scalar Import

enum Purpose {
  SECURITY
  EXECUTION
}

```
@dariuszkuc dariuszkuc added the change: patch Bug fixes label Sep 14, 2023
@dariuszkuc dariuszkuc merged commit c00e052 into apollographql:main Sep 14, 2023
3 checks passed
@dariuszkuc dariuszkuc deleted the link_purpose branch September 14, 2023 22:02
dariuszkuc added a commit that referenced this pull request Oct 10, 2024
Add missing `Purpose` enum. Full `@link` definition as per the
[spec](https://github.com/apollographql/specs/blob/main/link/v1.0/link-v1.0.graphql):

```graphql
directive @link(
    url: String!,
    as: String,
    import: [Import],
    for: Purpose)
repeatable on SCHEMA

scalar Import

enum Purpose {
  SECURITY
  EXECUTION
}
```

Resolves: #351
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
change: patch Bug fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

'link' directive definition is missing 'for' parameter
1 participant