docs: rewrite Label the incoming message with best practices#644
docs: rewrite Label the incoming message with best practices#644SangeetaMishr wants to merge 3 commits into
Conversation
…ractices The page was previously almost entirely a video embed that mostly covers other topics. Add an explanation of how the node actually works (it labels the contact's last incoming message), configuration steps, and the best practice of placing it right after a Wait for Response node so the label lands on the reply you actually want. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
|
Warning Review limit reached
Next review available in: 33 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThis change updates the "Label the incoming message" documentation page. The estimated read time was changed from 1 minute to 2 minutes. The page content was reorganized into new sections: "How it works," "Adding the node to your flow" with numbered steps and a screenshot, and expanded "Best practices" guidance. The previously embedded "Video Tutorial" section was removed and its YouTube iframe relocated to a new "Related video" section near the end of the page, separated by a divider. Changes
Sequence Diagram(s)No sequence diagram generated as this change is a documentation-only restructuring with no code or interaction flow to depict. Estimated code review effort: Medium Related issues: None mentioned Related PRs: None mentioned Suggested labels: documentation Suggested reviewers: None mentioned Poem: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
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-644--glific-docs.netlify.app |
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Summary
Fixes #614
The
Label the incoming messagepage was almost entirely a video embed, and per the issue, ~80% of that video isn't about this node - the part where it's actually used doesn't demonstrate correct usage either.This replaces it with:
Wait for Responsenode so the label lands on the reply you actually want, what happens if there's no incoming message yet (no-op), that multiple labels combine rather than overwrite, and to keep label names consistent for reporting.Verification
Confirmed the node's actual behavior against
glific/glific:lib/glific/flows/action.ex-add_input_labelsexecution labelscontext.last_messagelib/glific/processor/consumer_flow.ex-last_messageis set to the contact's message specifically incontinue_current_context, which runs when a flow resumes after the contact replies (i.e., right after aWait for Responsenode completes)add_flow_label/2no-ops whenlast_messageisnil, confirming the node does nothing if there's no incoming message to attach a label toTest plan
Summary by CodeRabbit