malloy icon indicating copy to clipboard operation
malloy copied to clipboard

HTML form: check for requires fields

Open Tectu opened this issue 3 years ago • 2 comments

The malloy::html::form class should perform check regarding all fields marked as required when parsing via parse().

Tectu avatar Aug 18 '22 14:08 Tectu

Draft: f86a48d189767b3e57028830738d23b5de2834a0

Tectu avatar Aug 21 '22 17:08 Tectu

I'm not too happy with this initial draft.

I think it might be better to have a separate/dedicated function in html::form to perform this check. This separate function could return/provide more detailed information such as the list of fields which are missing values. A consumer could then use this information to notify the user appropriately about the missing required fields.

Tectu avatar Aug 22 '22 11:08 Tectu