Skip to content

W-11904280 support GraphQL - #108

Open
IsaacEldridge wants to merge 2 commits into
latestfrom
W-11904280-support-graphql
Open

IsaacEldridge wants to merge 2 commits into
latestfrom
W-11904280-support-graphql

Conversation

@IsaacEldridge

Copy link
Copy Markdown

No description provided.

* GraphQL APIs
* Apollo GraphQL APIs

=== GraphQL Feature Support

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@gbattist I added those two tables here. We can remove the items that aren't supported

= Removing APIs from the Unified Schema

You can remove APIs from the unified schema as needed. For example, remove an API to add a new version, or remove an API if the underlying REST API no longer exists.
You can remove APIs from the unified schema as needed. For example, remove an API to add a new version, or remove an API if the underlying API no longer exists.

@gauravsgr gauravsgr Dec 13, 2022 •

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you think source API or underlying source API a better to use than underlying API? It matches with what has been updated in the other files in the PR @IsaacEldridge.

@gauravsgr gauravsgr left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good overall, added a cosmetic change recommendation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants