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

Conversation

@arvi3411301
Copy link
Member

Description

Hasura CLI should accept a json or yaml file for metadata apply command

Affected components

  • CLI

Related Issues

#1698

Solution and Design

CLI will check if metadata.yaml exists under migrations directory. If the same is not found, then metadata.json is checked.

If both are extensions are not present, then CLI will throw error.

FATA[0000] Unable to locate metadata.[yaml|json] file under migrations directory

Steps to test and verify

Export metadata from console (ie metadata.json) and place it under migrations directory.

Then, apply hasura metadata apply

@arvi3411301 arvi3411301 requested a review from shahidhk March 12, 2019 12:53
@arvi3411301 arvi3411301 added the c/cli Related to CLI label Mar 12, 2019
@hasura-bot
Copy link
Contributor

Review app for commit c02d951 deployed to Heroku: https://hge-ci-pull-1746.herokuapp.com
Docker image for server: hasura/graphql-engine:pull1746-c02d951

@shahidhk shahidhk changed the title cli: Accept a json or yaml file for metadata command (close #1698) accept a json or yaml file for metadata apply command (close #1698) Mar 18, 2019
@netlify
Copy link

netlify bot commented Mar 18, 2019

Deploy preview for hasura-docs ready!

Built with commit 72a466f

https://deploy-preview-1746--hasura-docs.netlify.com

@arvi3411301 arvi3411301 requested a review from rikinsk-zz as a code owner March 18, 2019 21:01
@hasura-bot
Copy link
Contributor

Review app for commit 72a466f deployed to Heroku: https://hge-ci-pull-1746.herokuapp.com
Docker image for server: hasura/graphql-engine:pull1746-72a466f

@shahidhk shahidhk merged commit 8043dde into hasura:master Mar 20, 2019
@hasura-bot
Copy link
Contributor

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

@arvi3411301 arvi3411301 deleted the issue-1698 branch March 20, 2019 08:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c/cli Related to CLI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants