Skip to content

[BHP1-1521] Fix Repeat Inputs Not Properly Cleared#182

Merged
Kcheung42 merged 3 commits intomainfrom
kc-BHP1-1521-fix-contain-summary-appearing
Mar 6, 2026
Merged

[BHP1-1521] Fix Repeat Inputs Not Properly Cleared#182
Kcheung42 merged 3 commits intomainfrom
kc-BHP1-1521-fix-contain-summary-appearing

Conversation

@Kcheung42
Copy link
Copy Markdown
Collaborator

@Kcheung42 Kcheung42 commented Feb 20, 2026

Purpose

  • Repeat Inputs (i.e. Resources) were not being properly cleared when the contain mode is switched.
  • Moved logic to look for groups instead of group-variables. This should fix the issue and maintain existing logic for non repeat inputs.

Related Issues

Closes BHP1-1521

Submission Checklist

  • Included Jira issue in the PR title (e.g. BHP1-### <title>)
  • Code passes linter rules (clj-kondo --lint components/**/src bases/**/src projects/**/src)
  • Feature(s) work when compiled (clojure -M:compile-cljs)

Testing

  1. Open this worksheet
    BHP1-1521.zip
    (has 2 blank resources when contain mode is "Add Resources") and re-run:
  2. Ensure Contain Summary Table now shows

Screenshots

@rjsheperd
Copy link
Copy Markdown
Contributor

Nice work! Approved 🎉

@Kcheung42 Kcheung42 merged commit 49e3a61 into main Mar 6, 2026
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