Conversation
WalkthroughThis 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)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
There was a problem hiding this comment.
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.
e217429 to
cd7973e
Compare
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
Summary by CodeRabbit