Skip to content

Comments

Update for SMS rates#2760

Merged
eldadfux merged 1 commit intomainfrom
feat-dns-changelog-update1
Feb 22, 2026
Merged

Update for SMS rates#2760
eldadfux merged 1 commit intomainfrom
feat-dns-changelog-update1

Conversation

@eldadfux
Copy link
Member

@eldadfux eldadfux commented Feb 22, 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

  • Documentation
    • Updated Phone OTP SMS pricing information, including per-country rate adjustments aligned with upstream providers.
    • Reorganized the pricing table for improved clarity across all supported destinations.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 22, 2026

No actionable comments were generated in the recent review. 🎉


Walkthrough

A new changelog entry file is added at src/routes/changelog/(entries)/2026-02-22.markdoc documenting SMS pricing updates for Phone OTP. The file contains YAML front matter specifying layout, title, and date metadata, followed by a body describing per-country rate adjustments, alignment with upstream providers, and a reference to Phone OTP documentation.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 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 'Update for SMS rates' is directly related to the changeset, which adds a changelog entry documenting SMS pricing updates for Phone OTP.
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 feat-dns-changelog-update1

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.

@eldadfux eldadfux merged commit 6c77764 into main Feb 22, 2026
6 checks passed
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.

1 participant