-
Notifications
You must be signed in to change notification settings - Fork 9
updatd the links to gcs set up, setting up the bucket and adding media #470
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
WalkthroughThis PR updates documentation references across three WhatsApp Groups Automation guides, replacing outdated URLs with new Pre Onboarding documentation paths and restructuring Prerequisites sections for clarity. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Suggested reviewers
Poem
Pre-merge checks and finishing touches❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✨ 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-470--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
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (3)
docs/6. WhatsApp Groups Automation/Google Sheets Integration For WhatsApp Groups.md(1 hunks)docs/6. WhatsApp Groups Automation/Onboarding for new NGOs.md(1 hunks)docs/6. WhatsApp Groups Automation/WhatsApp Groups Automation Features.md(1 hunks)
🧰 Additional context used
🪛 LanguageTool
docs/6. WhatsApp Groups Automation/Google Sheets Integration For WhatsApp Groups.md
[grammar] ~7-~7: There might be a mistake here.
Context: ... Prerequisites 1. Google cloud platform set up should be completed as per this [docume...
(QB_NEW_EN)
docs/6. WhatsApp Groups Automation/WhatsApp Groups Automation Features.md
[uncategorized] ~33-~33: “Google” is a proper noun and needs to be capitalized.
Context: ...te for sending media: 1. Should have a google cloud storage storage set up [refer her...
(A_GOOGLE)
[grammar] ~33-~33: There might be a mistake here.
Context: ...le%20Cloud%20Storage%20Setup%20-%20GCS). 2. Within the google cloud platform, should...
(QB_NEW_EN)
[uncategorized] ~34-~34: “Google” is a proper noun and needs to be capitalized.
Context: ...rage%20Setup%20-%20GCS). 2. Within the google cloud platform, should have a public bu...
(A_GOOGLE)
[uncategorized] ~35-~35: “Google” is a proper noun and needs to be capitalized.
Context: ...storage-bucket) 3. Upload files to the google cloud storage and generate the links [r...
(A_GOOGLE)
🔇 Additional comments (2)
docs/6. WhatsApp Groups Automation/Onboarding for new NGOs.md (1)
30-30: Link update looks good. Properly references the Pre Onboarding documentation path with correct URL encoding.docs/6. WhatsApp Groups Automation/Google Sheets Integration For WhatsApp Groups.md (1)
7-8: Prerequisites restructuring improves clarity. Splitting the setup steps into two enumerated items with correct anchor references to specific documentation sections is a good improvement for user navigation.
| 1. Should have a google cloud storage storage set up [refer here](https://glific.github.io/docs/docs/Pre%20Onboarding/Google%20Cloud%20Storage%20Setup%20-%20GCS). | ||
| 2. Within the google cloud platform, should have a public bucket setup [refer here](https://glific.github.io/docs/docs/Pre%20Onboarding/Google%20Cloud%20Storage%20Setup%20-%20GCS#step-4-create-a-storage-bucket) | ||
| 3. Upload files to the google cloud storage and generate the links [refer here](https://glific.github.io/docs/docs/Pre%20Onboarding/GCS%20Setup/Upload%20files%20on%20GCS) |
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 duplicate word and capitalization issues.
Three text quality issues need correction:
- Line 33: "google cloud storage storage set up" contains a duplicate "storage" word.
- Lines 33-35: "google" should be capitalized to "Google" (proper noun) in all three prerequisites.
Apply this diff:
-1. Should have a google cloud storage storage set up [refer here](https://glific.github.io/docs/docs/Pre%20Onboarding/Google%20Cloud%20Storage%20Setup%20-%20GCS).
-2. Within the google cloud platform, should have a public bucket setup [refer here](https://glific.github.io/docs/docs/Pre%20Onboarding/Google%20Cloud%20Storage%20Setup%20-%20GCS#step-4-create-a-storage-bucket)
-3. Upload files to the google cloud storage and generate the links [refer here](https://glific.github.io/docs/docs/Pre%20Onboarding/GCS%20Setup/Upload%20files%20on%20GCS)
+1. Should have a Google Cloud Storage set up [refer here](https://glific.github.io/docs/docs/Pre%20Onboarding/Google%20Cloud%20Storage%20Setup%20-%20GCS).
+2. Within the Google Cloud Platform, should have a public bucket setup [refer here](https://glific.github.io/docs/docs/Pre%20Onboarding/Google%20Cloud%20Storage%20Setup%20-%20GCS#step-4-create-a-storage-bucket)
+3. Upload files to Google Cloud Storage and generate the links [refer here](https://glific.github.io/docs/docs/Pre%20Onboarding/GCS%20Setup/Upload%20files%20on%20GCS)📝 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.
| 1. Should have a google cloud storage storage set up [refer here](https://glific.github.io/docs/docs/Pre%20Onboarding/Google%20Cloud%20Storage%20Setup%20-%20GCS). | |
| 2. Within the google cloud platform, should have a public bucket setup [refer here](https://glific.github.io/docs/docs/Pre%20Onboarding/Google%20Cloud%20Storage%20Setup%20-%20GCS#step-4-create-a-storage-bucket) | |
| 3. Upload files to the google cloud storage and generate the links [refer here](https://glific.github.io/docs/docs/Pre%20Onboarding/GCS%20Setup/Upload%20files%20on%20GCS) | |
| 1. Should have a Google Cloud Storage set up [refer here](https://glific.github.io/docs/docs/Pre%20Onboarding/Google%20Cloud%20Storage%20Setup%20-%20GCS). | |
| 2. Within the Google Cloud Platform, should have a public bucket setup [refer here](https://glific.github.io/docs/docs/Pre%20Onboarding/Google%20Cloud%20Storage%20Setup%20-%20GCS#step-4-create-a-storage-bucket) | |
| 3. Upload files to Google Cloud Storage and generate the links [refer here](https://glific.github.io/docs/docs/Pre%20Onboarding/GCS%20Setup/Upload%20files%20on%20GCS) |
🧰 Tools
🪛 LanguageTool
[uncategorized] ~33-~33: “Google” is a proper noun and needs to be capitalized.
Context: ...te for sending media: 1. Should have a google cloud storage storage set up [refer her...
(A_GOOGLE)
[grammar] ~33-~33: There might be a mistake here.
Context: ...le%20Cloud%20Storage%20Setup%20-%20GCS). 2. Within the google cloud platform, should...
(QB_NEW_EN)
[uncategorized] ~34-~34: “Google” is a proper noun and needs to be capitalized.
Context: ...rage%20Setup%20-%20GCS). 2. Within the google cloud platform, should have a public bu...
(A_GOOGLE)
[uncategorized] ~35-~35: “Google” is a proper noun and needs to be capitalized.
Context: ...storage-bucket) 3. Upload files to the google cloud storage and generate the links [r...
(A_GOOGLE)
🤖 Prompt for AI Agents
In docs/6. WhatsApp Groups Automation/WhatsApp Groups Automation Features.md
around lines 33 to 35, fix duplicate and capitalization errors: remove the
repeated word "storage" in "google cloud storage storage set up" and capitalize
"google" to "Google" in all three bullet points so each reads correctly (e.g.,
"Should have a Google Cloud Storage set up", "Within the Google Cloud Platform,
should have a public bucket set up", "Upload files to the Google Cloud Storage
and generate the links").
Summary by CodeRabbit
Documentation