Skip to content

Commit

Permalink
Merge pull request #74 from legalthings/conditional-required-issue
Browse files Browse the repository at this point in the history
Added fields to replicate problem with required field and condition
  • Loading branch information
svenstm authored Nov 27, 2017
2 parents 2d89297 + 000914e commit deec766
Showing 1 changed file with 24 additions and 3 deletions.
27 changes: 24 additions & 3 deletions js/demo/demo.js
Original file line number Diff line number Diff line change
Expand Up @@ -450,7 +450,7 @@
"label" : "Show next step",
"name" : "show_next_step",
"value" : "",
"helptext" : "",
"helptext" : "Type 'show' to see the next step",
"conditions" : "",
"pattern" : "",
"mask" : "",
Expand Down Expand Up @@ -481,12 +481,24 @@
"label" : "Show fields",
"name" : "show_fields",
"value" : "",
"helptext" : "",
"helptext" : "Type 'show' to see the fields",
"conditions" : "",
"pattern" : "",
"mask" : "",
"validation" : ""
},
{
"type" : "text",
"label" : "Required",
"name" : "required",
"value" : "",
"helptext" : "",
"conditions" : "conditional_step.show_fields != 'show'",
"required" : "required",
"pattern" : "",
"mask" : "",
"validation" : ""
},
{
"type" : "text",
"label" : "Masked field",
Expand Down Expand Up @@ -546,7 +558,16 @@
"conditions" : "validation.show_next_step == 'show'",
"helptext" : "",
"helptip" : ""
}
},
{
"fields" : [],
"label" : "Final step",
"group" : "final",
"article" : "",
"conditions" : "validation.show_next_step == 'show'",
"helptext" : "",
"helptip" : ""
}
]
};

Expand Down

0 comments on commit deec766

Please sign in to comment.