Skip to content

Check if response status is unprocessableEntity#31

Merged
marcelolx merged 2 commits intorails:mainfrom
olimart:patch-1
Aug 12, 2021
Merged

Check if response status is unprocessableEntity#31
marcelolx merged 2 commits intorails:mainfrom
olimart:patch-1

Conversation

@olimart
Copy link
Copy Markdown
Contributor

@olimart olimart commented Aug 11, 2021

This PR adds a convenient helper to check for unprocessableEntity response status.

I bet we don't want to add a helper for each status code but this is motivated by the fact that Rails now returns 422 status code by default for form submission with error rails/rails#41026

So, we can legitimately expect doing this kind of check more often.

@marcelolx
Copy link
Copy Markdown
Collaborator

Can you fix the linting issue?

@olimart
Copy link
Copy Markdown
Contributor Author

olimart commented Aug 12, 2021

Can you fix the linting issue?

Done. I think you need to approve the run of CI.

@marcelolx marcelolx merged commit 7179454 into rails:main Aug 12, 2021
@marcelolx
Copy link
Copy Markdown
Collaborator

Thanks!

@olimart olimart deleted the patch-1 branch August 12, 2021 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants