Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[instrument] norules meta instruction #8777

Closed
wants to merge 1 commit into from

Conversation

regisoc
Copy link
Contributor

@regisoc regisoc commented Jun 13, 2023

Brief summary of changes

Port #8714 to 24.1.

Testing instructions (if applicable)

  1. take one linst instrument.
  2. add a .meta file and/or add one line inside it: norules{@}true.
  3. go to the frontend > candidate list > choose a visit and timepoint with the corresponding instrument.
  4. try entering/modifying a form value with an empty answer (e.g. a blank value in a dropdown).
  5. This should save the form.
  6. Remove the norules{@}true from the meta file.
  7. try saving the form from the frontend again, it should make an error.

@regisoc regisoc self-assigned this Jun 13, 2023
@driusan
Copy link
Collaborator

driusan commented Jun 13, 2023

This is a new feature. Why would it go in 24.1?

@regisoc
Copy link
Contributor Author

regisoc commented Jun 13, 2023

For HBCD, linked to the optional fields issue of REDCap DD.

@regisoc
Copy link
Contributor Author

regisoc commented Jun 13, 2023

Will be put as an override in HBCD.

@regisoc regisoc closed this Jun 13, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants