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

feat: allow users to pass schema generator options for fed transforms #399

Merged
merged 1 commit into from
May 17, 2024
Merged

feat: allow users to pass schema generator options for fed transforms #399

merged 1 commit into from
May 17, 2024

Conversation

dariuszkuc
Copy link
Member

@dariuszkuc dariuszkuc commented May 17, 2024 •
edited
Loading

Adds following new transform method

  public static SchemaTransformer transform(
      final TypeDefinitionRegistry typeRegistry,
      final RuntimeWiring runtimeWiring,
      final SchemaGenerator.Options generatorOptions)

Resolves:
#163 and #389

@dariuszkuc
feat: allow users to pass schema generator options for fed transforma…
e48ee00 
…tions

Adds following new `transform` method

```java
  public static SchemaTransformer transform(
      final TypeDefinitionRegistry typeRegistry,
      final RuntimeWiring runtimeWiring,
      final SchemaGenerator.Options generatorOptions)
```
@dariuszkuc dariuszkuc added the change: minor New functionality added in backwards compatible manner label May 17, 2024
@dariuszkuc dariuszkuc requested a review from sachindshinde as a code owner May 17, 2024 05:43
@github-actions GitHub Actions
Copy link

Apollo Federation Subgraph Compatibility Results

Federation 1 Support Federation 2 Support
_service🟢
@key (single)🟢
@key (multi)🟢
@key (composite)🟢
repeatable @key🟢
@requires🟢
@provides🟢
federated tracing🟢
@link🟢
@shareable🟢
@tag🟢
@override🟢
@inaccessible🟢
@composeDirective🟢
@interfaceObject🟢

Learn more:

@dariuszkuc dariuszkuc merged commit 7f004b8 into main May 17, 2024
5 checks passed
@dariuszkuc dariuszkuc deleted the generate_schema_options branch May 17, 2024 05:47
@dariuszkuc dariuszkuc mentioned this pull request May 17, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sachindshinde sachindshinde Awaiting requested review from sachindshinde sachindshinde is a code owner

Assignees
No one assigned
Labels
change: minor New functionality added in backwards compatible manner
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@dariuszkuc