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

Changed url in config.yaml #772

Closed
wants to merge 1 commit into from
Closed

Conversation

vinayak42
Copy link
Contributor

@vinayak42 vinayak42 commented Oct 16, 2018

Resolve issue #751

I changed the existing url as required.
Fix #751
As this is my first PR to Hasura graph-ql engine, please mention further changes.

Description

What component does this PR affect?

  • Server
  • Console
  • CLI
  • Docs
  • Community Content
  • Build System

Requires changes from other components? If yes, please mark the components:

  • Server
  • Console
  • CLI
  • Docs
  • Community Content
  • Build System

Related Issue

Solution and Design

Type

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Docs update
  • Community content

Checklist:

  • I have read the contributing guide and my code conforms to the guidelines.
  • This change requires a change in the documentation.
  • I have updated the documentation accordingly.
  • I have added required tests.

@CLAassistant
Copy link

CLAassistant commented Oct 16, 2018

CLA assistant check
All committers have signed the CLA.

@shahidhk
Copy link
Member

@vinayak42 Thanks for the contribution. 🙏

But this change does not fix #751. It is more involved and require Go code changes. I will be commenting on the issue with more details.

Feel free to open another PR. 😄

@shahidhk shahidhk closed this Oct 16, 2018
@hasura-bot
Copy link
Contributor

Review app https://hge-ci-pull-772.herokuapp.com is deleted

hasura-bot pushed a commit that referenced this pull request Jun 27, 2024
### What

This adds a flag, `--partial-supergraph`, which instructs the metadata
resolver to prune relationships to unknown subgraphs rather than failing
to resolve.

### How

The flag gets passed through as
`metadata_resolve::configuration::Configuration`, and known subgraphs
are now tracked in `MetadataAccessor`. If the flag is set and a
relationship target refers to an unknown subgraph, we return an empty
list of relationships instead of failing.

Some test infrastructure has been added to set configuration flags per
test.

V3_GIT_ORIGIN_REV_ID: 6f0de2442a3bfc7c7a4c48e3dc7296dc1538cd67
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.

endpoint with trailing slash causes issues with cli
4 participants