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

Conversation

@rakeshkky
Copy link
Member

No description provided.

@rakeshkky rakeshkky added s/ok-to-merge Status: This pull request can be merged to master c/server Related to server labels Aug 7, 2018
@rakeshkky rakeshkky self-assigned this Aug 7, 2018
@rakeshkky rakeshkky requested a review from 0x777 August 7, 2018 10:17
stmtErr <- Q.getPGStmtErr txErr
codeMsg <- getPGCodeMsg stmtErr
extractError codeMsg
errTxt <- extractError codeMsg
Copy link
Member

Choose a reason for hiding this comment

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

Let's change extractError itself to return both code and message. We'll probably need to use the correct codes for the rest of the Postgres errors too

@hasura-bot
Copy link
Contributor

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

@hasura-bot
Copy link
Contributor

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

@hasura-bot
Copy link
Contributor

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

@hasura-bot
Copy link
Contributor

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

@hasura-bot
Copy link
Contributor

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

@hasura-bot
Copy link
Contributor

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

@shahidhk shahidhk merged commit adf973d into hasura:master Aug 10, 2018
@hasura-bot
Copy link
Contributor

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

hasura-bot pushed a commit that referenced this pull request Jan 8, 2024
V3_GIT_ORIGIN_REV_ID: 9eba8544b8401563b27c83f87e35b8448922b2f2
hasura-bot pushed a commit that referenced this pull request Jan 8, 2024
V3_GIT_ORIGIN_REV_ID: 9eba8544b8401563b27c83f87e35b8448922b2f2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c/server Related to server 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.

4 participants