feat(frontier): add acknowledge_token_forfeit to DeleteOrganizationRequest - #497
feat(frontier): add acknowledge_token_forfeit to DeleteOrganizationRequest#497whoAbhishekSah wants to merge 1 commit into
Conversation
…quest Deleting an organization can forfeit unused prepaid tokens on its billing account. The server needs the caller's consent inside the request to enforce this, so add a bool flag. When the account has unused tokens and the flag is not set, the delete fails with a failed_precondition error naming the amount at stake. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
The latest Buf updates on your PR. Results from workflow Validate / validate (pull_request).
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthrough
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
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. Comment |
What
Adds a
bool acknowledge_token_forfeitfield toDeleteOrganizationRequest.Why
Part of raystack/frontier#1837. Deleting an organization can forfeit unused prepaid tokens on its billing account. A UI confirmation popup is not enough on its own — the API is public (SDKs, scripts, curl), so the server can only enforce the consent if it arrives inside the request.
Planned behavior in frontier:
failed_preconditionerror saying how many tokens would be lost.🤖 Generated with Claude Code