-
Notifications
You must be signed in to change notification settings - Fork 9
Update Gupshup Setup with username retrieval instructions #564
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
Added instructions for retrieving Gupshup username.
📝 WalkthroughWalkthroughThis pull request updates the Gupshup Setup documentation file by refreshing the "Last Updated" metadata from October 2025 to January 2026 and adding a new FAQ entry explaining how to find your Gupshup username with step-by-step instructions. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 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-564--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/2`. Pre Onboarding/02. Gupshup Setup.md:
- Around line 246-255: Replace the plain email text "partner.support@gupshup.io"
with a mailto link, fix the nested list indentation to 2 spaces for the two
bullets under "Include the Following Details" (the org name and Gupshup app
name), and remove or soften the hard SLA phrase "The Gupshup team will reply
with your username and any guidance, within 24 hours." (e.g., change to "The
Gupshup team will typically reply with your username and guidance" or omit the
timeframe) so the doc passes MD034/MD007 lint rules and avoids an unsupported
promise.
| - Send an Email to partner.support@gupshup.io mark a copy to Glific Support Team (to help speed up the process) | ||
|
|
||
| - Include the Following Details | ||
|
|
||
| - Your organization name | ||
|
|
||
| - Your Gupshup app name (find it in your Glific instance: Settings → Gupshup) | ||
|
|
||
| The Gupshup team will reply with your username and any guidance, within 24 hours. | ||
|
|
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 markdown lint issues and avoid a hard SLA.
The email should be a mailto link (MD034), nested list indentation should be 2 spaces (MD007), and the “within 24 hours” promise is risky unless confirmed.
✅ Proposed edit
-- Send an Email to partner.support@gupshup.io mark a copy to Glific Support Team (to help speed up the process)
+- Send an email to [partner.support@gupshup.io](mailto:partner.support@gupshup.io) and CC the Glific Support Team (to help speed up the process)
- Include the Following Details
- - Your organization name
+ - Your organization name
- - Your Gupshup app name (find it in your Glific instance: Settings → Gupshup)
+ - Your Gupshup app name (find it in your Glific instance: Settings → Gupshup)
-The Gupshup team will reply with your username and any guidance, within 24 hours.
+The Gupshup team will reply with your username and any guidance.🧰 Tools
🪛 LanguageTool
[style] ~246-~246: Consider using “email”.
Context: ...p support team by email. Here’s how: - Send an Email to partner.support@gupshup.io mark a copy ...
(SEND_AN_EMAIL)
[style] ~246-~246: Consider using a different verb to strengthen your wording.
Context: ... a copy to Glific Support Team (to help speed up the process) - Include the Following D...
(SPEED_UP_ACCELERATE)
🪛 markdownlint-cli2 (0.20.0)
[warning] 246-246: Bare URL used
(MD034, no-bare-urls)
[warning] 250-250: Unordered list indentation
Expected: 2; Actual: 4
(MD007, ul-indent)
[warning] 252-252: Unordered list indentation
Expected: 2; Actual: 4
(MD007, ul-indent)
🤖 Prompt for AI Agents
In `@docs/2`. Pre Onboarding/02. Gupshup Setup.md around lines 246 - 255, Replace
the plain email text "partner.support@gupshup.io" with a mailto link, fix the
nested list indentation to 2 spaces for the two bullets under "Include the
Following Details" (the org name and Gupshup app name), and remove or soften the
hard SLA phrase "The Gupshup team will reply with your username and any
guidance, within 24 hours." (e.g., change to "The Gupshup team will typically
reply with your username and guidance" or omit the timeframe) so the doc passes
MD034/MD007 lint rules and avoids an unsupported promise.
| - Your Gupshup app name (find it in your Glific instance: Settings → Gupshup) | ||
|
|
||
| The Gupshup team will reply with your username and any guidance, within 24 hours. | ||
|
|
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.
this image is not visible on view mode, could you please check that
Added instructions for retrieving Gupshup username.
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.