-
Notifications
You must be signed in to change notification settings - Fork 9
Update 01. Facebook Verification Process for WhatsApp Business API.md #432
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update 01. Facebook Verification Process for WhatsApp Business API.md #432
Conversation
added more details on why and how images changed.
WalkthroughRewrote and expanded a single documentation page, replacing a video-centric tutorial with a structured, text-based, multi-section guide covering Facebook Business/Domain verification, prerequisites, step-by-step procedures, links, FAQs, and updated screenshots. Added metadata (read time, level, last updated) and references to third-party setup. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10–15 minutes Suggested reviewers
Poem
Pre-merge checks and finishing touches❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✨ 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 |
|
🚀 Deployed on https://deploy-preview-432--glific-docs.netlify.app |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (3)
docs/2. Pre Onboarding/01. Facebook Verification Process for WhatsApp Business API.md (3)
37-49: Fix list indentation under “Prerequisites”.The bullets at Lines 37-49 have extra leading spaces, triggering MD005/MD007 and causing inconsistent rendering. Please left-align the list items (no leading space before
-) so the markdown parser and our linter treat them as a single list.
177-179: Clean up the “Create a Business Manager” link.There’s a hard line break and extra spaces in the link text, which trips MD039 and can break the hyperlink. Keep the full link text on one line without internal padding (e.g.,
[Create a Business Manager](…)).
189-200: Wrap bare URLs in markdown links.Lines 189-200 and 239-252 expose bare URLs, violating MD034 and rendering inconsistently. Please wrap each in
[Label](https://…)form (e.g.,[business.facebook.com](https://business.facebook.com/)).Also applies to: 239-252
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
docs/2. Pre Onboarding/01. Facebook Verification Process for WhatsApp Business API.md(1 hunks)
🧰 Additional context used
🪛 markdownlint-cli2 (0.18.1)
docs/2. Pre Onboarding/01. Facebook Verification Process for WhatsApp Business API.md
39-39: Unordered list indentation
Expected: 0; Actual: 1
(MD007, ul-indent)
41-41: Inconsistent indentation for list items at the same level
Expected: 1; Actual: 0
(MD005, list-indent)
43-43: Inconsistent indentation for list items at the same level
Expected: 1; Actual: 0
(MD005, list-indent)
45-45: Inconsistent indentation for list items at the same level
Expected: 1; Actual: 0
(MD005, list-indent)
47-47: Inconsistent indentation for list items at the same level
Expected: 1; Actual: 0
(MD005, list-indent)
49-49: Inconsistent indentation for list items at the same level
Expected: 1; Actual: 0
(MD005, list-indent)
178-178: Spaces inside link text
(MD039, no-space-in-links)
189-189: Bare URL used
(MD034, no-bare-urls)
239-239: Bare URL used
(MD034, no-bare-urls)
240-240: Bare URL used
(MD034, no-bare-urls)
252-252: Bare URL used
(MD034, no-bare-urls)
added more details on why and how images changed.
Summary by CodeRabbit