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.

Sign up for GitHub

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

Jump to bottom

add server side validation for mandatory param #130

Open

azhamn opened this issue Apr 11, 2020 · 0 comments

Collaborator

azhamn commented Apr 11, 2020 •

edited

Loading

Currently in add alert/case apis we are not doing any validations. The mandatory checks are done from the client side (web portal).

We need to add server side validation as well for mandatory params: so far messages/info in english is required.
For the optional languages (si/ta) if the title is set, we must ensure the body is also present if not there will be incomplete info from user's pov

It's better to have a defined error response format as well (see - #131)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment