Skip to content

Comments

chore: update sms pricing table#2757

Merged
ChiragAgg5k merged 3 commits intomainfrom
update-sms-pricing-table
Feb 22, 2026
Merged

chore: update sms pricing table#2757
ChiragAgg5k merged 3 commits intomainfrom
update-sms-pricing-table

Conversation

@ChiragAgg5k
Copy link
Member

@ChiragAgg5k ChiragAgg5k commented Feb 20, 2026

Summary by CodeRabbit

  • Documentation
    • Updated SMS pricing in Phone OTP docs: replaced the long-form per-country entries with a condensed, headered pricing table, adjusted per-country Price/SMS values (increases and decreases), and streamlined country names and ordering to reflect current rates.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 20, 2026

Walkthrough

This pull request updates the SMS pricing section of a documentation page: the previous per-country Rates table is replaced with a reorganized header and an updated condensed table of Price / SMS (USD) values per country. Many country price entries were changed (increases, decreases, and some name/ordering adjustments). Changes are purely documentation data edits; no application logic, control flow, or exported/public entity signatures were modified.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 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 PR title 'chore: update sms pricing table' accurately and concisely describes the main change: updating SMS pricing data in a documentation table.
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 unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch update-sms-pricing-table

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: 3

🧹 Nitpick comments (1)
src/routes/docs/advanced/platform/phone-otp/+page.markdoc (1)

133-133: Outdated country name: "Macedonia" → "North Macedonia"

The country officially renamed itself to "North Macedonia" in 2019. Using the old name may cause confusion for users looking up this entry.

📝 Proposed fix
-| +389         | Macedonia                           | $ 0.04            |
+| +389         | North Macedonia                     | $ 0.04            |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@src/routes/docs/advanced/platform/phone-otp/`+page.markdoc at line 133,
Replace the outdated country name in the phone-OTP pricing table row that
currently reads "| +389         | Macedonia                           | $ 0.04  
|" with the official name "North Macedonia" so the row becomes "| +389         |
North Macedonia                     | $ 0.04            |"; search for the exact
string "Macedonia" (or the shown table row) and update it to "North Macedonia"
to keep the rest of the formatting intact.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@src/routes/docs/advanced/platform/phone-otp/`+page.markdoc:
- Around line 167-168: The two country rows are out of alphabetical order; swap
the Panama row ("| +507         | Panama                              | $ 0.13  
|") and the Pakistan row ("| +92          | Pakistan                           
| $ 0.36            |") so that the Pakistan row appears before the Panama row
in the table (i.e., reorder those two table lines to maintain alphabetical
ordering).
- Line 65: The table row currently "| +269         | Comoros and Mayotte        
| $ 0.29            |" incorrectly assigns Mayotte to +269; update the Markdoc
table so Comoros remains a +269 row (e.g., "| +269 | Comoros | $0.29 |") and
move or add Mayotte under the +262/Reunion row (or add a separate "| +262 |
Mayotte | $0.29 |" entry) by editing the "+269 ... Comoros and Mayotte" cell and
the existing "+262 Reunion" row in
src/routes/docs/advanced/platform/phone-otp/+page.markdoc so country codes and
labels match ITU assignments.
- Line 178: The table row that reads "| +7 | Russia, Kazakhstan and Turkmenistan
| $ 0.32 |" incorrectly attributes Turkmenistan to country code +7; update that
row to list only "Russia, Kazakhstan" for "+7" and add a new separate row for
Turkmenistan with country code "+993" and a placeholder rate (e.g., "$TBD" or
"price to be confirmed") so the table reflects +993 explicitly for Turkmenistan
and removes it from the +7 entry.

---

Nitpick comments:
In `@src/routes/docs/advanced/platform/phone-otp/`+page.markdoc:
- Line 133: Replace the outdated country name in the phone-OTP pricing table row
that currently reads "| +389         | Macedonia                           | $
0.04            |" with the official name "North Macedonia" so the row becomes
"| +389         | North Macedonia                     | $ 0.04            |";
search for the exact string "Macedonia" (or the shown table row) and update it
to "North Macedonia" to keep the rest of the formatting intact.

@ChiragAgg5k ChiragAgg5k force-pushed the update-sms-pricing-table branch from e217429 to cd7973e Compare February 20, 2026 16:20
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: 3

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@src/routes/docs/advanced/platform/phone-otp/`+page.markdoc:
- Line 133: Update the country display name for the +389 entry: replace
"Macedonia" with "North Macedonia" in the table row containing "| +389 |
Macedonia | $ 0.04 |" so the country name reflects the current official usage
while keeping the phone code and price unchanged.
- Line 198: Replace the outdated display name "Swaziland" with "Eswatini" in the
phone-code table row that contains the country code "+268" and the cost "$
0.11"; update the text exactly to "Eswatini" (preserve surrounding table
formatting and spacing) so the row reads "+268 | Eswatini | $ 0.11".
- Line 222: The table row currently listing "| +255         | Zanzibar          
| $ 0.11            |" is misleading because +255 is Tanzania's country code;
update the table to reflect the correct political/telecom assignment by either
replacing "Zanzibar" with "Tanzania (including Zanzibar)" or adding a separate
row for "Tanzania" (e.g., "| +255 | Tanzania (including Zanzibar) | $ 0.11 |")
so mainland Tanzania users are included and the entry is accurate.

---

Duplicate comments:
In `@src/routes/docs/advanced/platform/phone-otp/`+page.markdoc:
- Line 65: The table row '| +269         | Comoros and Mayotte      | $ 0.29    
|' is incorrect because Mayotte uses +262; update the table by changing the +269
row to reference only "Comoros" (e.g., '| +269 | Comoros | $0.29 |') and add
Mayotte under the +262 entry—either merge it into the existing '+262 Reunion'
row or add a distinct '+262 | Mayotte | ...' row with the correct pricing;
ensure the displayed country names and dialing codes are consistent across rows.

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: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@src/routes/docs/advanced/platform/phone-otp/`+page.markdoc:
- Around line 166-168: The country-code table rows are out of alphabetical
order: swap the Pakistan row ("Pakistan | +92 | $ 0.42") to come before the
Palau row ("Palau | +680 | $ 0.09") so the sequence reads Pakistan → Palau →
Panama; update the three table lines in +page.markdoc to place the Pakistan
entry above the Palau entry to maintain correct alphabetical ordering.

---

Duplicate comments:
In `@src/routes/docs/advanced/platform/phone-otp/`+page.markdoc:
- Line 65: Update the table row that currently reads "| +269         | Comoros
and Mayotte      | $ 0.34            |" so that the label no longer includes
Mayotte (change it to "Comoros"); also ensure there is a separate entry for
"+262" that includes Mayotte (and Réunion) if one does not already exist, so the
country-code labels are accurate.
- Line 198: Replace the outdated country name "Swaziland" with the current
official name "Eswatini" in the table row containing the string "| +268        
| Swaziland                | $ 0.13            |" in
src/routes/docs/advanced/platform/phone-otp/+page.markdoc; search for any other
occurrences of the literal "Swaziland" in that file and update them to
"Eswatini" while leaving the pricing/value unchanged.
- Line 133: The table row contains the outdated country name "Macedonia" —
replace the string "Macedonia" with "North Macedonia" in the row (the line
containing "| +389         | Macedonia                | $ 0.05            |")
and search the same file for any other occurrences of "Macedonia" to update them
consistently.

@ChiragAgg5k ChiragAgg5k merged commit 163e8c2 into main Feb 22, 2026
10 of 11 checks passed
@ChiragAgg5k ChiragAgg5k deleted the update-sms-pricing-table branch February 22, 2026 13:41
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