We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
When a pre-enrollment already exists the API returns a 404 status code. This code is not the most appropriate. It should be a 400 status code.
Originally posted by @mariajgrimaldi in #289 (comment)
The NotFound class by default has this status code:
NotFound
eox-core/eox_core/edxapp_wrapper/backends/pre_enrollment_l_v1.py
Line 45 in 7f13278
ValidationError
The text was updated successfully, but these errors were encountered:
AP-1405
No branches or pull requests
Description
When a pre-enrollment already exists the API returns a 404 status code. This code is not the most appropriate. It should be a 400 status code.
Originally posted by @mariajgrimaldi in #289 (comment)
Possible Solution
The
NotFound
class by default has this status code:eox-core/eox_core/edxapp_wrapper/backends/pre_enrollment_l_v1.py
Line 45 in 7f13278
Maybe using the
ValidationError
class (400) in this case is better.The text was updated successfully, but these errors were encountered: