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

Import by id now works by finding the relevant subgraph in the provided namespace #10

Merged
merged 5 commits into from
Nov 4, 2024

Conversation

SvenW
Copy link
Contributor

@SvenW SvenW commented Oct 22, 2024

The provider failed on importing subgraphs when provided only an id. The read part in the subgraph resource assumed that we always had a name to get the subgraph by

Copy link
Contributor

@Noroth Noroth left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@Aenimus Aenimus left a comment

Choose a reason for hiding this comment

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

LGTM!

@Noroth Noroth merged commit 58390b9 into wundergraph:main Nov 4, 2024
3 checks passed
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.

3 participants