-
Notifications
You must be signed in to change notification settings - Fork 9
Update 02. Send the contact a message.md #561
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
Conversation
Removed the incorrect link, added a few more details on sticker format
📝 WalkthroughWalkthroughUpdated the "Send the contact a message.md" documentation: refreshed Last Updated to January 2026; restructured and reworded Important Notes bullets; replaced a Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@docs/4`. Product Features/03. Flows/2. Flow Actions/02. Send the contact a
message.md:
- Around line 69-70: Fix the malformed list item and wrong URL in the two list
entries: close the first <li> that starts with "WhatsApp business API doesn't
support .gif files to be sent as attachments." by adding the missing </li>,
correct the link in the second list item that currently points to
"https://developers.facebook.com/docs/whatsapp/cloudapi/reference/media" to
"https://developers.facebook.com/docs/whatsapp/cloud-api/reference/media", and
add a space after the period so "follows.To" becomes "follows. To" to resolve
the MDX compilation error.
| <li>WhatsApp business API doesn't support .gif files to be sent as attachments. | ||
| <li>Size limits for attachments are as follows.To read more about this, please refer to this <a href="https://developers.facebook.com/docs/whatsapp/cloudapi/reference/media">link</a></li> |
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.
🧩 Analysis chain
🌐 Web query:
Facebook WhatsApp Cloud API media reference documentation URL
💡 Result:
Official Meta docs (WhatsApp Cloud API → Media reference):
https://developers.facebook.com/docs/whatsapp/cloud-api/reference/media/
(See sections like #upload-media, #retrieve-media-url, #download-media, #delete-media.) (stackoverflow.com)
Meta Postman reference (same endpoints, often easier to follow):
https://www.postman.com/meta/whatsapp-business-platform/request/fpj02x0/retrieve-media-url
https://www.postman.com/meta/whatsapp-business-platform/request/ked0afm/download-media
Citations:
- 1: https://stackoverflow.com/questions/74897702/whatsapp-cloud-api-can-not-retrieve-media-file?utm_source=openai
- 2: https://www.postman.com/meta/whatsapp-business-platform/request/fpj02x0/retrieve-media-url?utm_source=openai
Fix critical HTML syntax error and incorrect API documentation URL.
Line 69 is missing a closing </li> tag, which causes the MDX compilation to fail and blocks deployment.
Additionally, line 70 contains an incorrect URL path. The Facebook WhatsApp Cloud API documentation uses cloud-api (with hyphen), not cloudapi. The link should be https://developers.facebook.com/docs/whatsapp/cloud-api/reference/media.
Minor issues:
- Line 70 spacing:
follows.Toshould befollows. To
🐛 Proposed fix
- <li>WhatsApp business API doesn't support .gif files to be sent as attachments.
- <li>Size limits for attachments are as follows.To read more about this, please refer to this <a href="https://developers.facebook.com/docs/whatsapp/cloudapi/reference/media">link</a></li>
+ <li>WhatsApp business API doesn't support .gif files to be sent as attachments.</li>
+ <li>Size limits for attachments are as follows. To read more about this, please refer to this <a href="https://developers.facebook.com/docs/whatsapp/cloud-api/reference/media">link</a></li>📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| <li>WhatsApp business API doesn't support .gif files to be sent as attachments. | |
| <li>Size limits for attachments are as follows.To read more about this, please refer to this <a href="https://developers.facebook.com/docs/whatsapp/cloudapi/reference/media">link</a></li> | |
| <li>WhatsApp business API doesn't support .gif files to be sent as attachments.</li> | |
| <li>Size limits for attachments are as follows. To read more about this, please refer to this <a href="https://developers.facebook.com/docs/whatsapp/cloud-api/reference/media">link</a></li> |
🧰 Tools
🪛 GitHub Actions: Build and Deploy to Netlify
[error] 68-70: MDX compilation failed: Expected a closing tag for (69:5-69:9) before the end of paragraph. (MDX: end-tag-mismatch)
🪛 GitHub Actions: Test deployment
[error] 68-70: MDX compilation failed. Expected a closing tag for (69:5-69:9) before the end of paragraph. Cause: End-tag-mismatch (mdast-util-mdx-jsx).
🤖 Prompt for AI Agents
In `@docs/4`. Product Features/03. Flows/2. Flow Actions/02. Send the contact a
message.md around lines 69 - 70, Fix the malformed list item and wrong URL in
the two list entries: close the first <li> that starts with "WhatsApp business
API doesn't support .gif files to be sent as attachments." by adding the missing
</li>, correct the link in the second list item that currently points to
"https://developers.facebook.com/docs/whatsapp/cloudapi/reference/media" to
"https://developers.facebook.com/docs/whatsapp/cloud-api/reference/media", and
add a space after the period so "follows.To" becomes "follows. To" to resolve
the MDX compilation error.
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.
@CodeRabbit can you check fix or not ?
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.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Check the current state of lines 68-72 in the file
sed -n '68,72p' "docs/4. Product Features/03. Flows/2. Flow Actions/02. Send the contact a message.md"
Length of output: 498
@SangeetaMishr, the fix has not been applied yet. The file still has the critical issues:
Line 69 - Still has malformed HTML:
<li>WhatsApp business API doesn't support .gif files to be sent as attachments.<li>Should be:
<li>WhatsApp business API doesn't support .gif files to be sent as attachments.</li><li> needs to be changed to </li>
Line 70 - Still missing space:
<li>Size limits for attachments are as follows.To read more...Should be:
<li>Size limits for attachments are as follows. To read more...follows. To
These issues will continue to cause MDX compilation errors and block deployment until fixed. Please apply the suggested changes from the previous comment.
tanuprasad530
left a comment
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.
The url is line 70 is broken. This is the correct url - https://developers.facebook.com/docs/whatsapp/cloud-api/reference/media
kindly make the changes. The current one referenced exclude the - inside cloud-api
|
🚀 Deployed on https://deploy-preview-561--glific-docs.netlify.app |
|
@tanuprasad530 The issues has been fixed, you can review and merge by tuesday. |
Removed the incorrect link, added a few more details on sticker format
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.