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

LHC Form Builder and answer data type Boolean (Yes/No) #12

Closed
GitHubUNCStudent opened this issue Jul 25, 2019 · 6 comments
Closed

LHC Form Builder and answer data type Boolean (Yes/No) #12

GitHubUNCStudent opened this issue Jul 25, 2019 · 6 comments

Comments

@GitHubUNCStudent
Copy link

https://lhcformbuilder.nlm.nih.gov/ LHC Form Builder doesn't seem to support conditional show logic when the Source Field Answer Data Type is Boolean (Yes/No). We had to use a time-consuming workaround of changing all of the Answer Data Types (for Source Fields of Conditional Questions) to List with no exceptions (CNE) in order to get the conditional show logic to work. Will conditional show logic when the Source Field Answer Data Type is Boolean (Yes/No) functionality be added soon?

@plynchnlm
Copy link
Member

Thanks for raising this issue. We just took a look, and the skip logic seems to work if you require the source field boolean to be true, but not if you require it to be false. Is that the problem you were seeing? If you could provide the definition of a small form that demonstrates the problem you see, that would be helpful.

@GitHubUNCStudent
Copy link
Author

GitHubUNCStudent commented Jul 25, 2019 via email

@plynchnlm
Copy link
Member

I'm not seeing the attachment. It looks like GitHub supports only a limited number of file types. You might need to rename the .json files as .txt (changing the file extension). Alternatively, you could copy & paste the content into the comment. Just the LHC-Forms format is enough.

@GitHubUNCStudent
Copy link
Author

GitHubUNCStudent commented Jul 29, 2019 via email

@plynchnlm
Copy link
Member

Thanks for the example. Incidentally, you can add a second skip logic condition for "Will compensation be hourly" so that it will hide again if "Will compensation be provided to the study participants?" is changed back to false. (Just change the logic from "ANY" to "ALL")

@plynchnlm
Copy link
Member

The reported issue, that {value: false} does not work as a skipLogic trigger, is fixed (and probably has been for some time-- sorry for not updating this issue). The above LForms form definition also contains another issue, that it uses {value: null} in some places, and that is not supported and causes exceptions to be thrown. Instead, the supported syntax is {exists: false}.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants