[CRD-1915] feat: document chargeback file and evidence-type endpoints - #127
Open
ezerozen wants to merge 3 commits into
Open
[CRD-1915] feat: document chargeback file and evidence-type endpoints#127ezerozen wants to merge 3 commits into
ezerozen wants to merge 3 commits into
Conversation
Adds GET /files, GET /files/{file_id}, POST /files/batch, and GET
/evidence-types under /charges/{charge_id}/chargebacks/{chargeback_id},
matching bo-chargeback's gateway routes, which were previously undocumented.
Co-Authored-By: Claude Sonnet 5 <[email protected]>
Claude-Session: https://claude.ai/code/session_012862o2cEy3JbDvcNcG2sYb
|
ezerozen
marked this pull request as draft
August 19, 2026 18:48
Removes the unprecedented root Chargebacks tag entry (only Subscriptions domains get one), corrects the 400 error example's type/code to match bo-chargeback's actual error_handler.go mapping, and extracts the batch upload's multipart requestBody into requestBodies/chargebacks/ to match how every other non-trivial request body in this repo is organized. Co-Authored-By: Claude Sonnet 5 <[email protected]> Claude-Session: https://claude.ai/code/session_012862o2cEy3JbDvcNcG2sYb
ezerozen
marked this pull request as ready for review
August 20, 2026 17:17
|
ezerozen
changed the title
fcarrero
reviewed
Aug 20, 2026
The 16-value evidence type enum was duplicated across chargeback_evidence_file_response and chargeback_evidence_type_response; extracted to chargeback_evidence_type.yml and referenced via $ref from both, per review feedback from @fcarrero on PR #127. Co-Authored-By: Claude Sonnet 5 <[email protected]> Claude-Session: https://claude.ai/code/session_012862o2cEy3JbDvcNcG2sYb
fcarrero
approved these changes
Aug 20, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
bo-chargebackgateway endpoints under/charges/{charge_id}/chargebacks/{chargeback_id}:GET /files(getChargebackFiles)GET /files/{file_id}(getChargebackFile)POST /files/batch(uploadChargebackFilesBatch)GET /evidence-types(getChargebackEvidenceTypes)chargeback_evidence_file_response,chargeback_files_batch_response,chargeback_evidence_type_response) matching the actual Go response structs (gateway.UploadFileResponse,gateway.UploadBatchResponse,gateway.EvidenceTypeItem) — distinct from the pre-existingchargeback_response/chargeback_file_responseschemas, which model a different (payments-api) shape.charge_id,chargeback_id,file_id) and a new400.ymlerror resource (previously missing, needed since some business errors map to 400).POST /filesupload endpoint is intentionally left undocumented — out of scope for this change.Test plan
$refpaths verified to resolvemake mergerun via Docker — regenerated_build/api.yamlcleanly with no generator errors (initialadditionalPropertiesmultipart schema was replaced with explicit per-evidence-type properties after the generator flagged it as unsupported)openapi-generator-cli validate -i api.yaml— passes (only pre-existing unrelated warning: unusedevent_typesmodel)🤖 Generated with Claude Code
https://claude.ai/code/session_012862o2cEy3JbDvcNcG2sYb