bloom-housing / bloom

component-based web framework for affordable housing management
Apache License 2.0
34 stars 24 forks source link

fix: return proper exception for email in use #4241

Closed KrissDrawing closed 1 month ago

KrissDrawing commented 1 month ago

This PR addresses #770

Description

Returns 409 instead of 400 error for email in use (frontend already checks for 409 error code, and displays different message)

How Can This Be Tested/Reviewed?

In http://localhost:3001/users create user with already existing email (for example: admin@example.com). Error should display expected message

Author Checklist:

Review Process:

netlify[bot] commented 1 month ago

Deploy Preview for partners-bloom-dev ready!

Name Link
Latest commit d361b008c25dd48d3c78448458a527b06bc61250
Latest deploy log https://app.netlify.com/sites/partners-bloom-dev/deploys/66b4dd5b975f3e0008b6fa29
Deploy Preview https://deploy-preview-4241--partners-bloom-dev.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

netlify[bot] commented 1 month ago

Deploy Preview for bloom-exygy-dev ready!

Name Link
Latest commit d361b008c25dd48d3c78448458a527b06bc61250
Latest deploy log https://app.netlify.com/sites/bloom-exygy-dev/deploys/66b4dd5b0ebc970008673a5b
Deploy Preview https://deploy-preview-4241--bloom-exygy-dev.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.