Skip to content

Update SMS pricing table#3005

Merged
ChiragAgg5k merged 2 commits into
mainfrom
chore/update-sms-pricing-table
May 18, 2026
Merged

Update SMS pricing table#3005
ChiragAgg5k merged 2 commits into
mainfrom
chore/update-sms-pricing-table

Conversation

@ChiragAgg5k
Copy link
Copy Markdown
Member

@ChiragAgg5k ChiragAgg5k commented May 18, 2026

Summary

  • Updated the Phone OTP SMS rates table in the docs using the SMS pricing table generator.
  • Syncs website pricing with the Cloud SMS pricing config after updating the Cloud scraper to select the highest MSG91 route price when multiple routes are returned.
  • Only the rates table in the Phone OTP docs was changed.

Pricing table change summary

Metric Count
Countries in rates table 189
Changed displayed pricing entries 151
Unchanged displayed pricing entries 38
Displayed prices decreased 0
Displayed prices increased 151

Notable displayed increases

Country Old New Change
Mozambique 0.100 0.240 +140.0%
Venezuela 0.220 0.360 +63.6%
Senegal 0.290 0.380 +31.0%
Comoros and Mayotte 0.340 0.440 +29.4%
Madagascar 0.390 0.500 +28.2%

High-volume / reviewed countries

Country Old New
North America 0.009 0.010
Saudi Arabia 0.200 0.270
Yemen 0.250 0.280
Oman 0.180 0.210
Iraq 0.410 0.460
Jordan 0.420 0.470
Kuwait 0.220 0.240
Vietnam 0.210 0.230

Validation

  • Generated table with: php app/cli.php create-sms-pricing-table
  • Ran: git diff --check

@appwrite
Copy link
Copy Markdown

appwrite Bot commented May 18, 2026

Appwrite Website

Project ID: 69d7efb00023389e8d27

Sites (1)
Site Status Logs Preview QR
 website
69d7f2670014e24571ca
Failed Failed View Logs Preview URL QR Code

Website (appwrite/website)

Project ID: 684969cb000a2f6c0a02

Sites (1)
Site Status Logs Preview QR
 website
68496a17000f03d62013
Queued Queued View Logs Preview URL QR Code


Tip

GraphQL API works alongside REST and WebSocket protocols

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 18, 2026

Greptile Summary

This PR updates the Phone OTP SMS rates table in the docs to reflect the latest Cloud SMS pricing config, changing 124 of 189 country entries.

  • The actual diff shows the vast majority of prices increased (not decreased) relative to the old table — the PR description's "notable displayed decreases" table (Libya, Nigeria, Sri Lanka, Syria, Uzbekistan) is factually wrong; all five countries show price increases in the actual diff, and the magnitude of the increases listed in the "notable increases" section also understates the real diff values.
  • The PR description explicitly notes this table will need to be regenerated again once a follow-up Cloud PR (selecting the highest MSG91 route) is merged, meaning these prices are intentionally intermediate and will be superseded.

Confidence Score: 4/5

The file change itself is a straightforward data update to a Markdown pricing table with no structural issues, but the PR description's summary tables misrepresent the direction and magnitude of several price changes.

The pricing table in the file is internally consistent and follows the established format. However, the PR description's notable decreases section lists five countries that actually increased in price per the diff, and the notable increases table understates the actual new values. A reviewer approving based on the description alone would have a materially inaccurate understanding of what is being shipped.

src/routes/docs/advanced/platform/phone-otp/+page.markdoc — verify the new pricing values match the intended Cloud config snapshot before merging, given the acknowledged discrepancy between the PR description and the actual diff.

Important Files Changed

Filename Overview
src/routes/docs/advanced/platform/phone-otp/+page.markdoc SMS pricing table updated for 189 countries; 124 entries changed — nearly all prices increased rather than decreased, which conflicts with the PR description's "notable decreases" summary table

Reviews (2): Last reviewed commit: "Regenerate SMS pricing table" | Re-trigger Greptile

Comment thread src/routes/docs/advanced/platform/phone-otp/+page.markdoc
@ChiragAgg5k ChiragAgg5k merged commit 3406303 into main May 18, 2026
10 of 12 checks passed
@ChiragAgg5k ChiragAgg5k deleted the chore/update-sms-pricing-table branch May 18, 2026 12:24
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