Skip to content

Remove additional } in SMS payload template documentation#5874

Open
AnuradhaSK wants to merge 1 commit intomasterfrom
AnuradhaSK-patch-2
Open

Remove additional } in SMS payload template documentation#5874
AnuradhaSK wants to merge 1 commit intomasterfrom
AnuradhaSK-patch-2

Conversation

@AnuradhaSK
Copy link
Contributor

@AnuradhaSK AnuradhaSK commented Feb 8, 2026

Purpose

Screenshot 2026-02-08 at 22 57 09 This pull request makes a minor correction to the example JSON payload template for SMS notifications in the documentation. The change fixes a syntax error in the template example.
  • Documentation update:
    • Corrected the example JSON payload template in configure-sms-provider.md by removing an extra closing brace, ensuring the template is valid and easier to understand.

Summary by CodeRabbit

  • Documentation
    • Fixed a JSON syntax error in the SMS provider configuration guide. The example payload template for the Custom provider now displays proper JSON formatting, with a malformed closing brace removed to ensure users can accurately reference and implement the code snippet.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 8, 2026

Walkthrough

Fixed a syntactic error in a JSON payload template example within the SMS provider configuration documentation by removing an extraneous closing brace, improving the correctness of the template.

Changes

Cohort / File(s) Summary
Documentation Fix
en/includes/guides/notification-channels/configure-sms-provider.md
Removed extra closing brace from JSON payload example in the Custom provider section, correcting the template syntax.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A brace that lingered, now gone with glee,
The JSON now clean as can be!
🐰 With a hop and a bound, the fix was found,
Documentation's perfection all around! 🌟

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description check ⚠️ Warning The description covers the purpose of the fix with supporting context, but is missing required sections like Related PRs, Test environment, and Security checks from the template. Add the missing sections from the template: Related PRs, Test environment, and Security checks with appropriate responses or confirmations.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: removing an extra closing brace from the SMS payload template documentation.
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 AnuradhaSK-patch-2

No actionable comments were generated in the recent review. 🎉


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.

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