这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@arvi3411301
Copy link
Member

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.

@arvi3411301 arvi3411301 added the c/build-system Related to the build-release system label Jan 14, 2019
@hasura-bot
Copy link
Contributor

Review app for commit 6252179 deployed to Heroku: https://hge-ci-pull-1378.herokuapp.com
Docker image for server: hasura/graphql-engine:pull1378-6252179

Copy link
Member

@shahidhk shahidhk left a comment

Choose a reason for hiding this comment

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

LGTM

@shahidhk shahidhk changed the title remove sleep 60s for the console tests to start (close #1377) remove sleep for the console tests to start (close #1377) Jan 15, 2019
@shahidhk shahidhk merged commit a8dd49a into hasura:master Jan 15, 2019
@hasura-bot
Copy link
Contributor

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

@arvi3411301 arvi3411301 deleted the issue-1377 branch January 15, 2019 11:34
hasura-bot pushed a commit that referenced this pull request Nov 25, 2024
…1378)

<!-- The PR description should answer 2 important questions: -->

### What

<!-- What is this PR trying to accomplish (and why, if it's not
obvious)? -->

<!-- Consider: do we need to add a changelog entry? -->

<!-- Does this PR introduce new validation that might break old builds?
-->

<!-- Consider: do we need to put new checks behind a flag? -->
Note: Changelog not required as the relationships support in json:API is
not yet released.

In JSON:API - fix the output schema in the openapi schema generation by
including the components of the relationships schema.

### How

- Add functions to handle the relationship fields from the `ObjectType`
in schema generation code.
- Update the existing tests.

<!-- How is it trying to accomplish it (what are the implementation
steps)? -->

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

Labels

c/build-system Related to the build-release system

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants