Add CSV import/export functionality to task arguments dialog#1890
Add CSV import/export functionality to task arguments dialog#1890Mbeaulne wants to merge 1 commit into03-04-bulk_submissionsfrom
Conversation
🎩 To tophat this PR:You can add the following URL parameter to your browser to tophat this PR: |
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
bfeb63a to
c34183c
Compare
c34183c to
5b08c9a
Compare
9d5da6e to
afad2c5
Compare
src/components/shared/Submitters/Oasis/components/SubmitTaskArgumentsDialog.tsx
Outdated
Show resolved
Hide resolved
5b08c9a to
4528eaf
Compare
afad2c5 to
0d679b5
Compare
4528eaf to
38a9d1b
Compare
2a9fb0e to
a950f79
Compare
95ba6b3 to
e8f6fbd
Compare
camielvs
left a comment
There was a problem hiding this comment.
actually, changing my review because I want the raise the question: is this something we need?
This PR and subsequent ones are adding a bunch of complex methods, algorithms and other abstract logic and my gut sense is that there may be better alternatives
|
hmm. I see on Slack you mention you still need to clean up this stack. I will reserve judgment until that has been done. |
e8f6fbd to
6c31e06
Compare
465d436 to
46d35fc
Compare
6c31e06 to
97de98b
Compare
46d35fc to
8b94ba3
Compare
| } from "./componentSpec"; | ||
|
|
||
| /** | ||
| * RFC 4180 CSV parser. Handles quoted fields with commas and escaped quotes. |
There was a problem hiding this comment.
wondering if we can find npm package for this instead implementing ourselves?

Description
Added CSV import and export functionality to the task arguments dialog. Users can now download a CSV template with current argument values and import CSV files to populate pipeline inputs. The import supports both single-row (direct value assignment) and multi-row (bulk submission) formats. Secret inputs are automatically excluded from CSV operations for security. The system provides feedback on successful imports, unmatched columns, and skipped secret fields.
Related Issue and Pull requests
Type of Change
Checklist
Screenshots (if applicable)
bulk_sub.mov (uploaded via Graphite)
Test Instructions
Additional Comments
The CSV parser implements RFC 4180 standards, handling quoted fields with commas and escaped quotes. Comprehensive test coverage is included for all parsing and mapping functionality.