Skip to content

fix(api): support repeating groups in bulk submission updates#6644

Open
emaia wants to merge 1 commit intokobotoolbox:mainfrom
emaia:bulk-update-fix
Open

fix(api): support repeating groups in bulk submission updates#6644
emaia wants to merge 1 commit intokobotoolbox:mainfrom
emaia:bulk-update-fix

Conversation

@emaia
Copy link
Copy Markdown

@emaia emaia commented Jan 23, 2026

🗒️ Checklist

  1. run linter locally
  2. update developer docs (API, README, inline, etc.), if any
  3. for user-facing doc changes create a Zulip thread at #Support Docs Updates, if any
  4. draft PR with a title <type>(<scope>)<!>: <title> DEV-1234
  5. assign yourself, tag PR: at least Front end and/or Back end or workflow
  6. fill in the template below and delete template comments
  7. review thyself: read the diff and repro the preview as written
  8. open PR & confirm that CI passes & request reviewers, if needed
  9. delete this section before merging

📣 Summary

The PATCH /api/v2/assets/{asset_uuid}/data/bulk/ endpoint was returning 500 error when the payload contained repeating group fields (arrays). Modified edit_submission_xml() to handle list values by replacing existing repeat elements with new ones from the payload. Fix #6638

📖 Description

Modified edit_submission_xml() to handle list values by replacing existing repeat elements with new ones from the payload. Added helper functions _edit_repeat_group() and _extract_relative_path() to process nested repeat group structures recursively.

💭 Notes

  • Fix 500 error when bulk updating submissions with repeating group fields
  • Add support for array values in the bulk update payload to update repeat groups
  • Maintain backwards compatibility with existing simple field updates

@emaia emaia marked this pull request as ready for review January 23, 2026 13:51
@emaia emaia requested review from jnm and noliveleger as code owners January 23, 2026 13:51
@noliveleger noliveleger self-assigned this Jan 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

API bulk update returns status 500 when try update repeater fields

2 participants