Skip to content

Align ban partner API response with OpenAPI schema#3413

Merged
steven-tey merged 1 commit intomainfrom
fix-ban-partner-api-response
Feb 4, 2026
Merged

Align ban partner API response with OpenAPI schema#3413
steven-tey merged 1 commit intomainfrom
fix-ban-partner-api-response

Conversation

@devkiran
Copy link
Collaborator

@devkiran devkiran commented Feb 4, 2026

Summary by CodeRabbit

  • Improvements
    • Simplified the partner ban API response structure to return only the partner identifier, providing a more streamlined response format.

@vercel
Copy link
Contributor

vercel bot commented Feb 4, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
dub Ready Ready Preview Feb 4, 2026 3:46am

Request Review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 4, 2026

📝 Walkthrough

Walkthrough

The ban partners API endpoint was modified to simplify its response format, returning only a partnerId field instead of the full response object from the banPartner function call.

Changes

Cohort / File(s) Summary
Partner Ban API
apps/web/app/(ee)/api/partners/ban/route.ts
Simplified API response to return only partnerId instead of the full banPartner response object.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A partner banned, response so lean,
Just the ID, crisp and clean,
No clutter here, just what's needed,
Simplicity's virtue, well exceeded!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Align ban partner API response with OpenAPI schema' clearly describes the main change: modifying the API response to match the OpenAPI schema specification by returning only partnerId.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix-ban-partner-api-response

Important

Action Needed: IP Allowlist Update

If your organization protects your Git platform with IP whitelisting, please add the new CodeRabbit IP address to your allowlist:

  • 136.113.208.247/32 (new)
  • 34.170.211.100/32
  • 35.222.179.152/32

Reviews will stop working after February 8, 2026 if the new IP is not added to your allowlist.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@steven-tey steven-tey merged commit da4e3ff into main Feb 4, 2026
10 of 12 checks passed
@steven-tey steven-tey deleted the fix-ban-partner-api-response branch February 4, 2026 03:47
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