-
Notifications
You must be signed in to change notification settings - Fork 9
Sheet sync fails info #534
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
This document outlines common error conditions that can cause Google Sheets sync failures and provides solutions for each issue.
… Read from google sheets section
📝 WalkthroughWalkthroughDocumentation adds a "Google Sheet Sync" subsection to the Link Google Sheets guide (sync button, 24-hour auto-sync at 5–6am IST, multi-sync via FAQ link, failure behavior) and creates a new FAQ article enumerating five Google Sheets sync error scenarios with remediation steps. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related issues
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-534--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: 4
🧹 Nitpick comments (2)
docs/4. Product Features/03. Flows/2. Flow Actions/12. Link Google Sheets.md (1)
71-78: Minor grammar improvementsLine 76 contains a minor grammatical issue: "use case which demand" should be "use cases which demand" (or "use case which demands" in singular).
docs/8. FAQ/Why is the google sheet sync failing.md (1)
22-24: Clarify permission requirements and add contextThe permission description on line 22 could be clearer. It states that sheets should have "anyone with the link" can "view" permission, but this is specifically for read-only access. For clarity, consider:
- Specifying that this applies to read-only sheets.
- Clarifying what happens if write permissions are needed separately.
Additionally, line 24 should note that this is specifically for read-only sheets.
🔎 Proposed enhancement
-This happens if the sheet permissions are not in order. To use "read from google sheet" capability, the google sheet being used should have "anyone with the link" can "view" permission. +This happens if the sheet permissions are not in order. To use the "Read from Google Sheet" capability, the Google Sheet being used should have "Anyone with the link" can "View" permission (for read-only access).
📜 Review details
Configuration used: defaults
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
docs/4. Product Features/03. Flows/2. Flow Actions/12. Link Google Sheets.mddocs/8. FAQ/Why is the google sheet sync failing.md
🧰 Additional context used
🪛 LanguageTool
docs/8. FAQ/Why is the google sheet sync failing.md
[uncategorized] ~1-~1: “Google” is a proper noun and needs to be capitalized.
Context: # Why is the google sheet sync failing? Google sheets on ...
(A_GOOGLE)
[uncategorized] ~3-~3: “Google” is a proper noun and needs to be capitalized.
Context: ...rror condition that is displayed on the google sheets page. This document details ou...
(A_GOOGLE)
[uncategorized] ~5-~5: “Google” is a proper noun and needs to be capitalized.
Context: ... out the error conditions that make the google sheets sync to fail, and how to solve t...
(A_GOOGLE)
[uncategorized] ~14-~14: “Google” is a proper noun and needs to be capitalized.
Context: ...isplaying, it is likely because the the google sheet contains a very large amount of d...
(A_GOOGLE)
[style] ~14-~14: As an alternative to the over-used intensifier ‘very’, consider replacing this phrase.
Context: ...y because the the google sheet contains a very large amount of data. In case the google shee...
(EN_WEAK_ADJECTIVE)
[uncategorized] ~14-~14: “Google” is a proper noun and needs to be capitalized.
Context: ... very large amount of data. In case the google sheet contains a vast amount data, this...
(A_GOOGLE)
[uncategorized] ~14-~14: “Google” is a proper noun and needs to be capitalized.
Context: ...sheet contains a vast amount data, this google sheet has to be added as a table to the...
(A_GOOGLE)
[uncategorized] ~22-~22: “Google” is a proper noun and needs to be capitalized.
Context: ...ons are not in order. To use “read from google sheet” capability, the google sheet bei...
(A_GOOGLE)
[uncategorized] ~22-~22: “Google” is a proper noun and needs to be capitalized.
Context: ...read from google sheet” capability, the google sheet being used should have “anyone wi...
(A_GOOGLE)
[uncategorized] ~24-~24: Did you mean “Google Drive”?
Context: ...r in case the sheet is deleted from the google drive. ## 5. Repeated or missing headers Th...
(GOOGLE_PRODUCTS)
docs/4. Product Features/03. Flows/2. Flow Actions/12. Link Google Sheets.md
[uncategorized] ~73-~73: “Google” is a proper noun and needs to be capitalized.
Context: ... Sheet Sync - When content to a linked google sheet in read mode is updated, the shee...
(A_GOOGLE)
[uncategorized] ~73-~73: “Google” is a proper noun and needs to be capitalized.
Context: ...ng the sync button corresponding to the google sheet. - This ensures that the latest m...
(A_GOOGLE)
[uncategorized] ~74-~74: “Google” is a proper noun and needs to be capitalized.
Context: ...at the latest modifications made in the google sheet are avaiable to use in the Glific...
(A_GOOGLE)
[grammar] ~74-~74: Ensure spelling is correct
Context: ...ifications made in the google sheet are avaiable to use in the Glific flows where the sh...
(QB_NEW_EN_ORTHOGRAPHY_ERROR_IDS_1)
[uncategorized] ~75-~75: “Google” is a proper noun and needs to be capitalized.
Context: ... functionality sync every 24 hours. The google sync operations happen between 5-6am IS...
(A_GOOGLE)
[uncategorized] ~76-~76: “Google” is a proper noun and needs to be capitalized.
Context: ...o handle use case which demand that the google sheets be synced more than once a day o...
(A_GOOGLE)
[uncategorized] ~77-~77: “Google” is a proper noun and needs to be capitalized.
Context: ...8RvdO61wc2d4/edit?usp=sharing) - If the google sheet fails to sync, the flow continues...
(A_GOOGLE)
🪛 markdownlint-cli2 (0.18.1)
docs/8. FAQ/Why is the google sheet sync failing.md
14-14: Link text should be descriptive
(MD059, descriptive-link-text)
docs/4. Product Features/03. Flows/2. Flow Actions/12. Link Google Sheets.md
76-76: Link text should be descriptive
(MD059, descriptive-link-text)
| ### Google Sheet Sync | ||
|
|
||
| - When content to a linked google sheet in read mode is updated, the sheet needs to be `synced` using the sync button corresponding to the google sheet. | ||
| - This ensures that the latest modifications made in the google sheet are avaiable to use in the Glific flows where the sheet is being used. | ||
| - Google sheet in read-mode and on auto-sync functionality sync every 24 hours. The google sync operations happen between 5-6am IST | ||
| - To handle use case which demand that the google sheets be synced more than once a day or as per a custom logic, read more [here](https://docs.google.com/document/d/1Hix2ShKMJqfhOHrTc4GAT2JWc3ThyuZ8RvdO61wc2d4/edit?usp=sharing) | ||
| - If the google sheet fails to sync, the flow continues to send older content. | ||
| - To troubleshoot sheet sync failure errors, read more in FAQ section. |
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.
Capitalize "Google" consistently and improve link text clarity
The new Google Sheet Sync section has multiple issues:
- Capitalization: "google" should be capitalized as "Google" throughout (lines 73, 73, 74, 75, 76, 77).
- Link text: Line 76 uses "here" which is not descriptive. Consider: "follow the custom sync documentation" or similar.
- Reference clarity: Line 78 says "read more in FAQ section" but doesn't provide a direct link. Consider linking directly to the FAQ article (e.g.,
[Why is the google sheet sync failing?](../../../8.%20FAQ/Why%20is%20the%20google%20sheet%20sync%20failing.md)).
🔎 Proposed fixes
-### Google Sheet Sync
-
-- When content to a linked google sheet in read mode is updated, the sheet needs to be `synced` using the sync button corresponding to the google sheet.
-- This ensures that the latest modifications made in the google sheet are avaiable to use in the Glific flows where the sheet is being used.
-- Google sheet in read-mode and on auto-sync functionality sync every 24 hours. The google sync operations happen between 5-6am IST
-- To handle use case which demand that the google sheets be synced more than once a day or as per a custom logic, read more [here](https://docs.google.com/document/d/1Hix2ShKMJqfhOHrTc4GAT2JWc3ThyuZ8RvdO61wc2d4/edit?usp=sharing)
-- If the google sheet fails to sync, the flow continues to send older content.
-- To troubleshoot sheet sync failure errors, read more in FAQ section.
+### Google Sheet Sync
+
+- When content to a linked Google Sheet in read mode is updated, the sheet needs to be `synced` using the sync button corresponding to the Google Sheet.
+- This ensures that the latest modifications made in the Google Sheet are available to use in the Glific flows where the sheet is being used.
+- Google Sheets in read-mode and on auto-sync functionality sync every 24 hours. The Google sync operations happen between 5-6am IST.
+- To handle use cases which demand that the Google Sheets be synced more than once a day or as per a custom logic, refer to [this custom sync guide](https://docs.google.com/document/d/1Hix2ShKMJqfhOHrTc4GAT2JWc3ThyuZ8RvdO61wc2d4/edit?usp=sharing).
+- If the Google Sheet fails to sync, the flow continues to send older content.
+- To troubleshoot sheet sync failure errors, refer to [Why is the google sheet sync failing?](../../../8.%20FAQ/Why%20is%20the%20google%20sheet%20sync%20failing.md).🧰 Tools
🪛 LanguageTool
[uncategorized] ~73-~73: “Google” is a proper noun and needs to be capitalized.
Context: ... Sheet Sync - When content to a linked google sheet in read mode is updated, the shee...
(A_GOOGLE)
[uncategorized] ~73-~73: “Google” is a proper noun and needs to be capitalized.
Context: ...ng the sync button corresponding to the google sheet. - This ensures that the latest m...
(A_GOOGLE)
[uncategorized] ~74-~74: “Google” is a proper noun and needs to be capitalized.
Context: ...at the latest modifications made in the google sheet are avaiable to use in the Glific...
(A_GOOGLE)
[grammar] ~74-~74: Ensure spelling is correct
Context: ...ifications made in the google sheet are avaiable to use in the Glific flows where the sh...
(QB_NEW_EN_ORTHOGRAPHY_ERROR_IDS_1)
[uncategorized] ~75-~75: “Google” is a proper noun and needs to be capitalized.
Context: ... functionality sync every 24 hours. The google sync operations happen between 5-6am IS...
(A_GOOGLE)
[uncategorized] ~76-~76: “Google” is a proper noun and needs to be capitalized.
Context: ...o handle use case which demand that the google sheets be synced more than once a day o...
(A_GOOGLE)
[uncategorized] ~77-~77: “Google” is a proper noun and needs to be capitalized.
Context: ...8RvdO61wc2d4/edit?usp=sharing) - If the google sheet fails to sync, the flow continues...
(A_GOOGLE)
🪛 markdownlint-cli2 (0.18.1)
76-76: Link text should be descriptive
(MD059, descriptive-link-text)
🤖 Prompt for AI Agents
In docs/4. Product Features/03. Flows/2. Flow Actions/12. Link Google Sheets.md
around lines 71 to 78, capitalize all instances of "google" to "Google", replace
the vague link text "here" with a descriptive phrase like "follow the custom
sync documentation" (and update the anchor to that text), and replace "read more
in FAQ section" with a direct link to the FAQ article (for example: [Why is the
Google Sheet sync
failing?](../../../8.%20FAQ/Why%20is%20the%20Google%20Sheet%20sync%20failing.md))
so readers have clear, specific links and consistent capitalization.
docs/4. Product Features/03. Flows/2. Flow Actions/12. Link Google Sheets.md
Outdated
Show resolved
Hide resolved
…ogle Sheets.md Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
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
♻️ Duplicate comments (1)
docs/4. Product Features/03. Flows/2. Flow Actions/12. Link Google Sheets.md (1)
71-78: Apply previously-flagged capitalization and link text improvements.This subsection still contains the exact issues flagged in the previous review (commit 35195db): lowercase "google" throughout, vague link text ("here"), and an indirect FAQ reference instead of a direct link. These corrections are essential for consistency, accessibility, and discoverability.
🔎 Proposed fix (from previous review)
### Google Sheet Sync -- When content to a linked google sheet in read mode is updated, the sheet needs to be `synced` using the sync button corresponding to the google sheet. -- This ensures that the latest modifications made in the google sheet are available to use in the Glific flows where the sheet is being used. -- Google sheet in read-mode and on auto-sync functionality sync every 24 hours. The google sync operations happen between 5-6am IST -- To handle use case which demand that the google sheets be synced more than once a day or as per a custom logic, read more [here](https://docs.google.com/document/d/1Hix2ShKMJqfhOHrTc4GAT2JWc3ThyuZ8RvdO61wc2d4/edit?usp=sharing) -- If the google sheet fails to sync, the flow continues to send older content. -- To troubleshoot sheet sync failure errors, read more in FAQ section. +- When content to a linked Google Sheet in read mode is updated, the sheet needs to be `synced` using the sync button corresponding to the Google Sheet. +- This ensures that the latest modifications made in the Google Sheet are available to use in the Glific flows where the sheet is being used. +- Google Sheets in read-mode and on auto-sync functionality sync every 24 hours. The Google sync operations happen between 5-6am IST. +- To handle use cases which demand that the Google Sheets be synced more than once a day or as per a custom logic, refer to [this custom sync guide](https://docs.google.com/document/d/1Hix2ShKMJqfhOHrTc4GAT2JWc3ThyuZ8RvdO61wc2d4/edit?usp=sharing). +- If the Google Sheet fails to sync, the flow continues to send older content. +- To troubleshoot sheet sync failure errors, refer to [Why is the Google Sheet sync failing?](../../../8.%20FAQ/Why%20is%20the%20google%20sheet%20sync%20failing.md).
📜 Review details
Configuration used: defaults
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
docs/4. Product Features/03. Flows/2. Flow Actions/12. Link Google Sheets.mddocs/8. FAQ/Why is the google sheet sync failing.md
🧰 Additional context used
🪛 LanguageTool
docs/4. Product Features/03. Flows/2. Flow Actions/12. Link Google Sheets.md
[uncategorized] ~73-~73: “Google” is a proper noun and needs to be capitalized.
Context: ... Sheet Sync - When content to a linked google sheet in read mode is updated, the shee...
(A_GOOGLE)
[uncategorized] ~73-~73: “Google” is a proper noun and needs to be capitalized.
Context: ...ng the sync button corresponding to the google sheet. - This ensures that the latest m...
(A_GOOGLE)
[uncategorized] ~74-~74: “Google” is a proper noun and needs to be capitalized.
Context: ...at the latest modifications made in the google sheet are available to use in the Glifi...
(A_GOOGLE)
[uncategorized] ~75-~75: “Google” is a proper noun and needs to be capitalized.
Context: ... functionality sync every 24 hours. The google sync operations happen between 5-6am IS...
(A_GOOGLE)
[uncategorized] ~76-~76: “Google” is a proper noun and needs to be capitalized.
Context: ...o handle use case which demand that the google sheets be synced more than once a day o...
(A_GOOGLE)
[uncategorized] ~77-~77: “Google” is a proper noun and needs to be capitalized.
Context: ...8RvdO61wc2d4/edit?usp=sharing) - If the google sheet fails to sync, the flow continues...
(A_GOOGLE)
docs/8. FAQ/Why is the google sheet sync failing.md
[uncategorized] ~3-~3: “Google” is a proper noun and needs to be capitalized.
Context: ...rror condition that is displayed on the google sheets page. This document details ou...
(A_GOOGLE)
[uncategorized] ~5-~5: “Google” is a proper noun and needs to be capitalized.
Context: ... out the error conditions that make the google sheets sync to fail, and how to solve t...
(A_GOOGLE)
[uncategorized] ~22-~22: “Google” is a proper noun and needs to be capitalized.
Context: ...ons are not in order. To use “read from google sheet” capability, the google sheet bei...
(A_GOOGLE)
[uncategorized] ~22-~22: “Google” is a proper noun and needs to be capitalized.
Context: ...read from google sheet” capability, the google sheet being used should have “anyone wi...
(A_GOOGLE)
[uncategorized] ~24-~24: Did you mean “Google Drive”?
Context: ...r in case the sheet is deleted from the google drive. ## 5. Repeated or missing headers Th...
(GOOGLE_PRODUCTS)
🪛 markdownlint-cli2 (0.18.1)
docs/4. Product Features/03. Flows/2. Flow Actions/12. Link Google Sheets.md
76-76: Link text should be descriptive
(MD059, descriptive-link-text)
🔇 Additional comments (1)
docs/8. FAQ/Why is the google sheet sync failing.md (1)
1-35: Article structure and troubleshooting content are well-organized.The five error scenarios are clearly documented with practical remediation steps and visual references. Line 14's descriptive link text ("this BigQuery and Cloud Run integration guide") provides good accessibility and context. The document effectively complements the Google Sheet Sync section in the Link Google Sheets guide by offering concrete error diagnosis and resolution steps.
| @@ -0,0 +1,35 @@ | |||
| # Why is the Google Sheet sync failing? | |||
|
|
|||
| Google sheets on read mode with auto-sync functionality turned on sync once in every 24 hours. The sync operations happens around 5-6 am IST. If google sheet fails to sync, there is an error condition that is displayed on the google sheets page. | |||
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.
Capitalize "Google" and "Google Drive" throughout the document.
Several instances of lowercase "google" should be capitalized as "Google" (a proper noun). Additionally, line 24 should use "Google Drive" (the official product name) instead of "google drive". Line 3 also contains a grammar issue: "happens" should be "happen" (plural subject).
🔎 Proposed fixes
-Google sheets on read mode with auto-sync functionality turned on sync once in every 24 hours. The sync operations happens around 5-6 am IST. If google sheet fails to sync, there is an error condition that is displayed on the google sheets page.
+Google Sheets on read mode with auto-sync functionality turned on sync once in every 24 hours. The sync operations happen around 5-6 am IST. If the Google Sheet fails to sync, there is an error condition that is displayed on the Google Sheets page.-This document details out the error conditions that make the google sheets sync to fail, and how to solve these situations are explained below.
+This document details the error conditions that make the Google Sheets sync fail, and how to solve these situations.-This happens if the sheet permissions are not in order. To use "read from google sheet" capability, the google sheet being used should have "anyone with the link" can "view" permission.
+This happens if the sheet permissions are not in order. To use "read from Google Sheet" capability, the Google Sheet being used should have "anyone with the link" can "view" permission.-This condition could also occur in case the sheet is deleted from the google drive.
+This condition could also occur in case the sheet is deleted from Google Drive.Also applies to: 5-5, 22-22, 24-24
🧰 Tools
🪛 LanguageTool
[uncategorized] ~3-~3: “Google” is a proper noun and needs to be capitalized.
Context: ...rror condition that is displayed on the google sheets page. This document details ou...
(A_GOOGLE)
🤖 Prompt for AI Agents
In docs/8. FAQ/Why is the google sheet sync failing.md around lines 3, 5, 22 and
24, fix capitalization and a grammar issue: change all instances of "google" to
"Google", change "google drive" to "Google Drive" on line 24, and correct
"happens" to "happen" on line 3; ensure consistency across the file by searching
for any other lowercase "google" and replacing with the proper product names.
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.