Skip to content

Add stategic metadata to insert_submission, SubmissionCompleted and SubmissionFailed#58

Merged
OpsBotPrime merged 5 commits intomasterfrom
jerbaroo-42
Feb 17, 2026
Merged

Add stategic metadata to insert_submission, SubmissionCompleted and SubmissionFailed#58
OpsBotPrime merged 5 commits intomasterfrom
jerbaroo-42

Conversation

@jerbaroo
Copy link
Contributor

@jerbaroo jerbaroo commented Feb 5, 2026

Add stategic metadata as a parameter to insert_submission (which just passes it to insert_submission_chunks), and make the strategic metadata available in the submission results: SubmissionCompleted and SubmissionFailed.

@jerbaroo jerbaroo linked an issue Feb 5, 2026 that may be closed by this pull request
@jerbaroo jerbaroo changed the title Add stategic metadata to SubmissionCompleted andSubmissionFailed Add stategic metadata to insert_submission, SubmissionCompleted and SubmissionFailed Feb 9, 2026
@jerbaroo jerbaroo marked this pull request as ready for review February 9, 2026 08:31
@jerbaroo jerbaroo requested a review from Qqwy February 9, 2026 08:33
Copy link
Contributor

@Qqwy Qqwy left a comment

Choose a reason for hiding this comment

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

Hey Jeremy! A very nice PR so far. The one request I have is that you add a test for both of the cases (to the Python test suite).

@jerbaroo jerbaroo force-pushed the jerbaroo-42 branch 3 times, most recently from aa4f7ba to d5db03b Compare February 10, 2026 11:17
@jerbaroo jerbaroo requested a review from Qqwy February 10, 2026 11:47
@jerbaroo jerbaroo force-pushed the jerbaroo-42 branch 3 times, most recently from 8486574 to ac3e448 Compare February 10, 2026 14:56
@jerbaroo
Copy link
Contributor Author

@OpsBotPrime merge and tag

…leted and SubmissionFailed

Approved-by: jerbaroo
Priority: Normal
Auto-deploy: false
@OpsBotPrime
Copy link
Contributor

Rebased as 7371353, waiting for CI …

@OpsBotPrime
Copy link
Contributor

CI job 🟡 started.

@OpsBotPrime
Copy link
Contributor

@jerbaroo I tagged your PR with v44. Please wait for the build of 7371353 to pass and don't forget to deploy it!

@OpsBotPrime OpsBotPrime merged commit 7371353 into master Feb 17, 2026
6 of 7 checks passed
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.

Support fetching a submission's metadata from the producer side

3 participants