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

[LINST] skip metadatafields for surveys #8961

Merged
merged 3 commits into from
Nov 24, 2023

Conversation

zaliqarosli
Copy link
Contributor

@zaliqarosli zaliqarosli commented Nov 16, 2023

Brief summary of changes

replacing #8863

Skip over metadatafields and calculate ages in surveys

This is not a big change, one way or another the fields were either being skipped with an empty check, through a project override or through code in the module itself handling the submission. this is just an explicit check to make everything clearer

  • Have you updated related documentation?

Testing instructions (if applicable)

Link(s) to related issue(s)

  • Resolves # (Reference the issue this fixes, if any.)

@zaliqarosli zaliqarosli added the Priority: Projects PR or issue is a priority for at least one project and should be a higher priority for LORIS label Nov 16, 2023
@driusan driusan merged commit c1f25c9 into aces:24.1-release Nov 24, 2023
9 checks passed
@ridz1208 ridz1208 added this to the 24.1.6 milestone Apr 9, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Priority: Projects PR or issue is a priority for at least one project and should be a higher priority for LORIS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants