Skip to content

Conversation

@lohanidamodar
Copy link
Member

@lohanidamodar lohanidamodar commented Jan 21, 2026

What does this PR do?

(Provide a description of what this PR does.)

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

(Write your answer here.)

Summary by CodeRabbit

  • Bug Fixes
    • Fixed region listing in billing to display organization-specific regions instead of global regions, ensuring accurate region availability for each organization's billing context.

✏️ Tip: You can customize this high-level summary in your review settings.

@appwrite
Copy link

appwrite bot commented Jan 21, 2026

Console (appwrite/console)

Project ID: 688b7bf400350cbd60e9

Sites (1)
Site Status Logs Preview QR
 console-stage
688b7cf6003b1842c9dc
Ready Ready View Logs Preview URL QR Code

Tip

MCP server integration brings LLM superpowers to Claude Desktop and Cursor IDE

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 21, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Walkthrough

The change modifies the API endpoint used by the Billing.listRegions method in the billing SDK. The endpoint path shifts from /console/regions (global console path) to /organizations/{organizationId}/regions (organization-scoped path). The HTTP method (GET) and parameter structure remain unchanged. This is a single-line modification affecting only the target endpoint URL.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 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 'Fix: regions endpoint' is directly related to the changeset, which modifies the region-listing API path in the Billing.listRegions function.
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.


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.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
src/lib/sdk/billing.ts (1)

1410-1424: Correct HTTP method capitalization for consistency.

The endpoint change correctly scopes region listing to the organization level and is a non-breaking change. However, the HTTP method should use lowercase 'get' to match the pattern of similar organization-scoped methods like listInvoices (line 797), which uses lowercase. Change line 1417 from 'GET' to 'get' for consistency.

@ItzNotABug ItzNotABug marked this pull request as draft January 21, 2026 14:09
@lohanidamodar lohanidamodar marked this pull request as ready for review January 21, 2026 14:34
@lohanidamodar lohanidamodar merged commit cb737e6 into main Jan 21, 2026
9 checks passed
@lohanidamodar lohanidamodar deleted the fix-regions branch January 21, 2026 14:34
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.

3 participants