Form fields validation, Events and Providers

PrototypeK02 commented 2 years ago

Description

Please include a summary of the change.

Events create validation fields Provider create validation fields

Screenshots

If applicable, add screenshots about your changes

Type of change

Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How to test it

If applicable, describe the tests that you ran to verify your changes. Provide instructions so we can reproduce.

Go to '...'
Click on '....'
Scroll down to '....'

Checklist:

[ ] Doesn't break the current code.
[ ] Passes linters and test, also is building.
[ ] Doesn't have spelling or grammatical problems.
[ ] Doesn't have unnecessary comments or debugging code.
[ ] The branch is updated with the last dev branch changes.

vercel[bot] commented 2 years ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated
did-admin-panel	✅ Ready (Inspect)	Visit Preview	Sep 19, 2022 at 6:22AM (UTC)

e-azocar commented 2 years ago

@PrototypeK02 please check this out

PrototypeK02 commented 2 years ago

Seems like all the fields are required now?

@Eghinner please check

Also, remove the lockfile change, please

Yes, the custom input component makes the input required, but it wasnt working as a boolean, by default the required option for the form was false, and the property required from the custom input was a string.

angelik0828 commented 2 years ago

:heavy_check_mark: Verification performed found

-The fields, being empty and clicking save, throw the error written below the text field. - events

In providers, clicking add also shows me the fields with a text below the text field.

Azordev / did-admin-panel