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

RA-2037, update only the fields within the specified question #139

Merged
merged 1 commit into from
Jun 3, 2024

Conversation

cioan
Copy link
Member

@cioan cioan commented Jun 3, 2024

@mogoodrich , I have added the comment you requested about the 'mother-field' field. I have also updated the jQuery statement to only update the fields within the configured question. The relationship widget had two instances configured in out SL registration form, one for capturing the mother info, and another instance to capture the children. Therefore there were multiple fields with the name 'other_person_uuid' and that caused problems. So now I restrict the jQuery select statement to the field name within the given question name.

@cioan cioan requested a review from mogoodrich June 3, 2024 14:37
Copy link
Member

@mogoodrich mogoodrich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks @cioan !

@cioan
Copy link
Member Author

cioan commented Jun 3, 2024

Thanks @mogoodrich !

@cioan cioan merged commit a567365 into master Jun 3, 2024
1 check passed
@cioan cioan deleted the RA-2037 branch June 3, 2024 21:48
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

Successfully merging this pull request may close these issues.

2 participants