-
Notifications
You must be signed in to change notification settings - Fork 9
Update last updated date and add video content section #565
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 last updated date and added a section for sending video content via HSM.
📝 WalkthroughWalkthroughDocumentation update to HSM Templates file: timestamp refreshed to January 2026, new section added describing HSM video content workflows with follow-up messaging, and explanatory guidance added for applying HSM messages across multiple languages using consistent element naming. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
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)
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-565--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/06. HSM Templates.md:
- Line 233: Fix the grammar and spacing in the HSM Templates doc: replace the
comma-splice line "Add a Wait Time, Use Wait for Time (e.g., 3 hours) instead of
Wait for Response." with two sentences "Add a Wait Time. Use Wait for Time
(e.g., 3 hours) instead of Wait for Response." and remove the extra space before
the colon in the phrase "with button :" so it reads "with button:"; locate these
strings in the document to update them.
🧹 Nitpick comments (1)
docs/4. Product Features/06. HSM Templates.md (1)
223-239: Consider adding visual examples for consistency.Other sections in this document include screenshots to illustrate the UI and workflow. Adding similar visual aids here (e.g., showing the HSM setup with video link button, the Wait for Time configuration) would improve consistency and user comprehension.
Additionally, consider linking to related documentation for referenced features like "Wait for Time" functionality.
|
|
||
| - Add a button with the video link, but note that clicks on this button won’t be recorded, as the link opens outside and does not send a response back to the flow. | ||
|
|
||
| - Add a Wait Time, Use Wait for Time (e.g., 3 hours) instead of Wait for Response. |
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.
Fix grammatical and formatting issues.
Two minor issues:
- Line 233: Comma splice creates a run-on sentence. Consider: "Add a Wait Time. Use Wait for Time (e.g., 3 hours) instead of Wait for Response."
- Line 235: Remove extra space before the colon: "with button:" (not "with button :")
📝 Proposed fix
-- Add a Wait Time, Use Wait for Time (e.g., 3 hours) instead of Wait for Response.
+- Add a Wait Time. Use Wait for Time (e.g., 3 hours) instead of Wait for Response.
-- Send an approved HSM message with button : _Did you complete the activity?_
+- Send an approved HSM message with button: _Did you complete the activity?_Also applies to: 235-235
🤖 Prompt for AI Agents
In `@docs/4`. Product Features/06. HSM Templates.md at line 233, Fix the grammar
and spacing in the HSM Templates doc: replace the comma-splice line "Add a Wait
Time, Use Wait for Time (e.g., 3 hours) instead of Wait for Response." with two
sentences "Add a Wait Time. Use Wait for Time (e.g., 3 hours) instead of Wait
for Response." and remove the extra space before the colon in the phrase "with
button :" so it reads "with button:"; locate these strings in the document to
update them.
Updated last updated date and added a section for sending video content via HSM.
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.