Skip to content

fix(realunit): align sell endpoint Swagger doc with Level 30 enforcement#3652

Open
TaprootFreak wants to merge 1 commit intodevelopfrom
fix/realunit-sell-kyc-level-doc
Open

fix(realunit): align sell endpoint Swagger doc with Level 30 enforcement#3652
TaprootFreak wants to merge 1 commit intodevelopfrom
fix/realunit-sell-kyc-level-doc

Conversation

@TaprootFreak
Copy link
Copy Markdown
Collaborator

Summary

  • Controller Swagger doc advertised KYC Level 20 but service enforces Level 30 → users on Level 20–29 got 403/400 despite the docs.
  • Updates @ApiOperation.description and @ApiBadRequestResponse.description on PUT /v1/realunit/sell to reflect the actual Level 30 requirement.
  • Pure documentation alignment — no behavior change, no DTO change, no migration.

Closes #3649

Test plan

  • npm run format:check green
  • npm run type-check green
  • npm run lint green
  • Swagger UI for PUT /v1/realunit/sell shows "Requires KYC Level 30" in operation description and 400-response description
  • Behavior unchanged: Level 30 user can quote a sell, Level 29 user gets KycLevelRequiredException with message KYC Level 30 required for RealUnit sell

The controller advertised KYC Level 20 while the service enforces Level 30,
causing confusion for users on Level 20-29 who got 403/400 despite the docs.
Updates the operation description and 400-response description to reflect
the actual Level 30 requirement.

Closes #3649
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.

RealUnit sell: KYC level mismatch between controller doc and service behavior

2 participants