-
Notifications
You must be signed in to change notification settings - Fork 9
Revise HSM Template Message Error documentation #563
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
base: main
Are you sure you want to change the base?
Conversation
Updated the HSM Template Message Error documentation to include new troubleshooting steps and key points regarding message delivery issues and frequency capping.
📝 WalkthroughWalkthroughDocumentation updates to HSM Template Message Error guide: refreshed the introductory header metadata and added a new troubleshooting section detailing Meta's frequency capping requirements, dynamic blocking mechanisms, and recommended retry wait periods with external resource references. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Suggested reviewers
Poem
🚥 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)
Tip 🧪 Unit Test Generation v2 is now available!We have significantly improved our unit test generation capabilities. To enable: Add this to your reviews:
finishing_touches:
unit_tests:
enabled: trueTry it out by using the Have feedback? Share your thoughts on our Discord thread! 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-563--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: 1
🤖 Fix all issues with AI agents
In `@docs/4`. Product Features/10. Others/HSM Template Message Error.md:
- Around line 45-46: The documentation currently links the anchor text "Per-User
Marketing Template Message Limits" to the Facebook "error-codes" page, which is
incorrect; update the link in the sentence that contains the Gupshup blog and
the "[Per-User Marketing Template Message Limits]" anchor so it either (a)
replaces the URL
(https://developers.facebook.com/documentation/business-messaging/whatsapp/support/error-codes)
with the correct Facebook Developers page that documents per-user
marketing/template message limits for WhatsApp, or (b) change the anchor text to
accurately describe the linked page (e.g., "WhatsApp error codes") if you intend
to keep the error-codes URL; ensure the anchor text and target URL match.
🧹 Nitpick comments (2)
docs/4. Product Features/10. Others/HSM Template Message Error.md (2)
1-9: Consider improving table accessibility.The header table lacks semantic markup for accessibility. Consider using
<th>elements for the header row or adding ARIA labels to improve screen reader support.♿ Suggested accessibility improvements
<h3> <table> <tr> - <td><b>10 minutes read</b></td> - <td style={{ paddingLeft: '40px' }}><b>Level: Beginner </b></td> - <td style={{ paddingLeft: '40px' }}><b>Last Updated: January 2026</b></td> + <th scope="col">10 minutes read</th> + <th scope="col" style={{ paddingLeft: '40px' }}>Level: Beginner</th> + <th scope="col" style={{ paddingLeft: '40px' }}>Last Updated: January 2026</th> </tr> </table> </h3>
48-48: Remove hardcoded image dimensions for responsive design.The image has fixed
width="921" height="717"attributes that may not scale well on mobile devices or different screen sizes. Consider removing these attributes and letting CSS handle responsive sizing.📱 Proposed fix for responsive images
-<img width="921" height="717" alt="image" src="https://github.com/user-attachments/assets/e897a19f-1c41-4913-9052-f5025cfc5e7e" /> +<img alt="image" src="https://github.com/user-attachments/assets/e897a19f-1c41-4913-9052-f5025cfc5e7e" />
| - For details on WhatsApp’s frequency capping rules, see [Gupshup Blog: All You Need to Know About WhatsApp Frequency Capping](https://www.gupshup.ai/resources/blog/all-you-need-to-know-about-whatsapp-business-api-frequency-capping/) and check the | ||
| [Per-User Marketing Template Message Limits](https://developers.facebook.com/documentation/business-messaging/whatsapp/support/error-codes) |
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
# Verify that the external URLs are accessible and return valid responses
echo "Checking Gupshup blog URL..."
curl -I -s -o /dev/null -w "HTTP Status: %{http_code}\n" "https://www.gupshup.ai/resources/blog/all-you-need-to-know-about-whatsapp-business-api-frequency-capping/"
echo -e "\nChecking Facebook developers URL..."
curl -I -s -o /dev/null -w "HTTP Status: %{http_code}\n" "https://developers.facebook.com/documentation/business-messaging/whatsapp/support/error-codes"Repository: glific/docs
Length of output: 154
🏁 Script executed:
#!/bin/bash
# Fetch the Facebook URL content to verify it covers "Per-User Marketing Template Message Limits"
echo "Fetching Facebook developers page content..."
curl -s "https://developers.facebook.com/documentation/business-messaging/whatsapp/support/error-codes" | head -c 3000Repository: glific/docs
Length of output: 3100
🏁 Script executed:
#!/bin/bash
# Search for mentions of "marketing template" or "per-user" in the error codes page
echo "Checking for marketing template or per-user limits content..."
curl -s "https://developers.facebook.com/documentation/business-messaging/whatsapp/support/error-codes" | grep -i -E "(marketing|per-user|template.*limit|frequency.*cap)" | head -20Repository: glific/docs
Length of output: 50368
Correct the Facebook developers link—it points to error codes, not template message limits.
The Facebook developers link is accessible but incorrect. The page titled "Error codes" (documenting WhatsApp Business Platform Cloud API error codes and solutions) does not cover "Per-User Marketing Template Message Limits" as stated in the documentation. Either replace the link with the correct Facebook developers page covering template message limits, or update the text to reflect what the error-codes page actually contains. The Gupshup blog link is correctly accessible.
🤖 Prompt for AI Agents
In `@docs/4`. Product Features/10. Others/HSM Template Message Error.md around
lines 45 - 46, The documentation currently links the anchor text "Per-User
Marketing Template Message Limits" to the Facebook "error-codes" page, which is
incorrect; update the link in the sentence that contains the Gupshup blog and
the "[Per-User Marketing Template Message Limits]" anchor so it either (a)
replaces the URL
(https://developers.facebook.com/documentation/business-messaging/whatsapp/support/error-codes)
with the correct Facebook Developers page that documents per-user
marketing/template message limits for WhatsApp, or (b) change the anchor text to
accurately describe the linked page (e.g., "WhatsApp error codes") if you intend
to keep the error-codes URL; ensure the anchor text and target URL match.
Updated the HSM Template Message Error documentation to include new troubleshooting steps and key points regarding message delivery issues and frequency capping.
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.