Skip to content
This repository has been archived by the owner on Jan 7, 2022. It is now read-only.

Check required fields in update event type form #26

Closed
a1exsh opened this issue Sep 18, 2018 · 1 comment
Closed

Check required fields in update event type form #26

a1exsh opened this issue Sep 18, 2018 · 1 comment
Labels
help wanted Extra attention is needed

Comments

@a1exsh
Copy link
Contributor

a1exsh commented Sep 18, 2018

Currently the form for updating an event type would happily allow you to omit some fields which are marked as required in the form itself, e.g. "Owning application" and "Audience". Then the API responds with 400 Bad Request which is sometimes not really straightforward to parse.

In case of omitting the owning application, no error occurs at all. So this is contradictory to the marking of the field as required.

Preferably, the HTML form in UI would highlight the required and missing fields with red color when the user tries to submit it, avoiding the API call in the first place.

@SergKam SergKam added the help wanted Extra attention is needed label Sep 18, 2018
@SergKam
Copy link
Contributor

SergKam commented Nov 20, 2018

Fixed

@SergKam SergKam closed this as completed Nov 20, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants