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

Conversation

@praveenweb
Copy link
Member

No description provided.

@praveenweb praveenweb requested a review from karthikvt26 July 27, 2018 15:05
@karthikvt26
Copy link
Contributor

@arvi3411301 review app for this pr?

@praveenweb praveenweb added s/ok-to-merge Status: This pull request can be merged to master c/console Related to console labels Jul 28, 2018
@arvi3411301
Copy link
Member

@karthikvt26 status check for server changed from test_and_build_server to build_server.

@shahidhk can we change it back to test_and_build_server?

@hasura-bot
Copy link
Contributor

Review app available at: https://hge-ci-pull-220.herokuapp.com

@karthikvt26
Copy link
Contributor

karthikvt26 commented Jul 28, 2018

@praveenweb I tested the following usecases

  1. Create table

While creating table I encountered a tiny css issue

screenshot 391

  1. Saving comment by leaving the input field empty shows [object object] as output

screenshot 392

  1. After I clicked on edit, I removed the [object object] from the input results in a tiny CSS issue.

screenshot 393

  1. Unable to clear added comment under columns

screenshot 394

screenshot 395

  1. Can we escape the comment text? If I start with ' it doesn't let me ( It might not be super important )

screenshot 396

Let me know

@hasura-bot
Copy link
Contributor

Review app available at: https://hge-ci-pull-220.herokuapp.com

if (state.tableComment && state.tableComment !== '') {
sqlCreateTable +=
' COMMENT ON TABLE ' +
currentSchema +
Copy link
Contributor

Choose a reason for hiding this comment

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

Space before this

@arvi3411301 arvi3411301 merged commit 2e58483 into hasura:master Jul 30, 2018
@hasura-bot
Copy link
Contributor

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

@arvi3411301
Copy link
Member

/heroku deploy

@hasura-bot
Copy link
Contributor

Review app available at: https://hge-ci-pull-220.herokuapp.com

@arvi3411301
Copy link
Member

/heroku delete

@hasura-bot
Copy link
Contributor

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

@arpit4
Copy link

arpit4 commented Jul 31, 2018

/heroku deploy

@hasura-bot
Copy link
Contributor

@arpit4 you don't have enough permissions to execute this command

@praveenweb praveenweb deleted the issue-206 branch May 21, 2019 06:58
polRk pushed a commit to polRk/graphql-engine that referenced this pull request Feb 12, 2020
hgiasac pushed a commit to hgiasac/graphql-engine that referenced this pull request Dec 19, 2023
GitOrigin-RevId: e78e8f7f2810b0b1ef7a34cd7aace49c1e559a8c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c/console Related to console s/ok-to-merge Status: This pull request can be merged to master

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants