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

Conversation

@AnnieAtHasura
Copy link
Contributor

@AnnieAtHasura AnnieAtHasura commented Jun 11, 2019

Description

User will now see controls for add/modify/delete event triggers and remote schemas when in migration mode.

Affected components

  • Console

Related Issues

close #1875

Solution and Design

Removed check for migrationmode when not needed

Steps to test and verify

set migration mode on,
visit the events trigger page, add a simple event trigger, modify it, then delete it
visit the remote schemas page, use the Glitch example to make a remote schema,
modify it, and delete it.
visit the Data page and see that you can't modify the DDL

Limitations, known bugs & workarounds

You can still trick the system by entering SQL

@netlify
Copy link

netlify bot commented Jun 11, 2019

Deploy preview for hasura-docs ready!

Built with commit 56b9936

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

@hasura-bot
Copy link
Contributor

Review app for commit 0308891 deployed to Heroku: https://hge-ci-pull-2366.herokuapp.com
Docker image for server: hasura/graphql-engine:pull2366-03088916

@strongSoda strongSoda added the c/console Related to console label Jun 11, 2019
@hasura-bot
Copy link
Contributor

Review app for commit cf0857c deployed to Heroku: https://hge-ci-pull-2366.herokuapp.com
Docker image for server: hasura/graphql-engine:pull2366-cf0857c5

@rikinsk-zz rikinsk-zz merged commit 034c550 into hasura:master Jun 13, 2019
@hasura-bot
Copy link
Contributor

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

@hasura-bot
Copy link
Contributor

Review app for commit 56b9936 deployed to Heroku: https://hge-ci-pull-2366.herokuapp.com
Docker image for server: hasura/graphql-engine:pull2366-56b99365

@wawhal
Copy link
Contributor

wawhal commented Jun 14, 2019

@arvi3411301 this review app is not deleted even after the PR is merged. FYI.

@arvi3411301
Copy link
Member

@wawhal This is because the PR was merged even before the review app was deployed for the status check.

polRk pushed a commit to polRk/graphql-engine that referenced this pull request Feb 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c/console Related to console

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Enable event triggers & remote schemas even if migrations mode is off

8 participants