Skip to content

chore(launchpad): Return created instead of OK upon chunks being successfully uploaded #97559

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

NicoHinderling
Copy link
Contributor

A request made by @szokeasaurusrex so that we're more idiomatically correct even though we aren't ultimately going to return "OK" state anymore. This will be landed after we land a change to the CLI to be satisfied by the endpoint when it receives "CREATED" instead of "OK" (current functionality).

@NicoHinderling NicoHinderling requested a review from a team as a code owner August 10, 2025 19:02
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Aug 10, 2025
Copy link

codecov bot commented Aug 10, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #97559      +/-   ##
==========================================
- Coverage   80.73%   80.62%   -0.11%     
==========================================
  Files        8566     8560       -6     
  Lines      378963   376889    -2074     
  Branches    24538    24538              
==========================================
- Hits       305948   303867    -2081     
- Misses      72645    72652       +7     
  Partials      370      370              

@NicoHinderling
Copy link
Contributor Author

actually just going to merge this into #97558

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Scope: Backend Automatically applied to PRs that change backend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant