Skip to content
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

Server error when trying to create Box without a purpose #1910

Closed
ysbaddaden opened this issue Feb 9, 2023 · 1 comment · Fixed by #1914
Closed

Server error when trying to create Box without a purpose #1910

ysbaddaden opened this issue Feb 9, 2023 · 1 comment · Fixed by #1914
Assignees
Labels

Comments

@ysbaddaden
Copy link
Contributor

ysbaddaden commented Feb 9, 2023

Steps to reproduce:

Try to create a box without selecting a purpose and the server will respond with an INTERNAL SERVER ERROR message.

Culprits

  1. we don't display the error to the user alongside the form input (aka 'unhandled form error');
  2. the mechanism to report this warning to sentry crashes, we thus end-up reporting the crash to sentry instead of the unhandled form parameters (oops).

Log:

Unhandled form errors in Box: {:purpose=>["Purpose can't be blank", "Purpose is not included in the list"]}

Sentry issue RADX-CDX-10

@sardar-usman
Copy link

@ysbaddaden It's showing a proper error now. Working as expected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants