Skip to content
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.

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

Fix record list validation #577

Merged
merged 1 commit into from Feb 25, 2020
Merged

Fix record list validation #577

merged 1 commit into from Feb 25, 2020

Conversation

sanjacornelius
Copy link
Contributor

@sanjacornelius sanjacornelius commented Feb 24, 2020

Changes

- Add validation to the Add Record Modal fields.

24 February, 2020 - Loom Recording - Watch Video

To Test

  • Create a screen with a Record List inside
  • Create a page with required fields
  • Associate this page to the Record List
  • Associate this screen to a task in a process
  • Run a case

Expected Behavior
If the Record List page has a required field before adding the record, it validates the required field, preventing adding of empty rows.

closes #550

@sanjacornelius sanjacornelius added the review Review this ticket label Feb 24, 2020
@velkymx velkymx merged commit 74a84a3 into develop Feb 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
review Review this ticket
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Validation Broken in Record List
2 participants