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

fix error code for when insertion check constraint fails (fix #257) #267

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 7 commits into from
Aug 10, 2018

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
simplifyError txErr = do
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