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

modify comments for tables/columns and views #220

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

Merged
merged 5 commits into from
Jul 30, 2018

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