Skip to content

Add submissionStatus to bulk-submit kickoff#63

Merged
elsaperelli merged 1 commit intomainfrom
submissionStatus-kickoff
Dec 3, 2025
Merged

Add submissionStatus to bulk-submit kickoff#63
elsaperelli merged 1 commit intomainfrom
submissionStatus-kickoff

Conversation

@elsaperelli
Copy link
Collaborator

@elsaperelli elsaperelli commented Dec 2, 2025

Summary

This PR adds the submissionStatus parameter to the $bulk-submit request parameters for the $bulk-submit kickoff functionality. It also removes the '-' characters from submitter value so that it works with deqm-test-server. Must be tested with this PR once this change gets made.

New behavior

There should be no new behavior.

Code changes

  • changes bulk-export-submitter to bulkExportSubmitter
  • adds submissionStatus to the request parameters

Testing guidance

  • npm run check
  • npm run start on PORT 3001 and test with deqm-test-server on PORT 3000
  • Run through the bulk-submit kickoff workflows (same as before, but update submitter value to bulkExportSubmitter)

@github-actions
Copy link

github-actions bot commented Dec 2, 2025

Coverage report

St.
Category Percentage Covered / Total
🟡 Statements 73.95% 596/806
🟡 Branches 62.87% 215/342
🟡 Functions 70.99% 93/131
🟡 Lines 74.3% 587/790

Test suite run success

97 tests passing in 8 suites.

Report generated by 🧪jest coverage report action from 5ac1aa7

@lmd59 lmd59 self-requested a review December 3, 2025 14:06
@lmd59 lmd59 self-assigned this Dec 3, 2025
Copy link
Contributor

@lmd59 lmd59 left a comment

Choose a reason for hiding this comment

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

Looks good!

@elsaperelli elsaperelli merged commit d894a59 into main Dec 3, 2025
4 checks passed
@elsaperelli elsaperelli deleted the submissionStatus-kickoff branch December 3, 2025 19:51
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