-
Notifications
You must be signed in to change notification settings - Fork 8
Create Use of Google Sheets To Update Contact Variables.md #305
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
Uploaded the summary of work for done for SArC which lists 4 use cases of using google sheets and contact variables and custom expressions.
WalkthroughA new documentation file has been added that explains how Google Sheets can be integrated with AI assistants for the Saturday Art Class program. The document details four distinct use cases: assigning unique keywords for chatbot interactions, managing sequential art class plans, employing fuzzy matching for keyword corrections, and organizing media submissions in Google Cloud based on contact variables. Changes
Sequence Diagram(s)sequenceDiagram
participant Teacher
participant Chatbot
participant GoogleSheets
participant CloudStorage
Teacher->>Chatbot: Sends keyword for art class plan
Chatbot->>GoogleSheets: Request plan details (keywords, counters, flags)
GoogleSheets-->>Chatbot: Returns plan data
Chatbot->>Teacher: Delivers art class plan
Teacher->>Chatbot: Submits activity & media
Chatbot->>Chatbot: Fuzzy match to validate keyword
Chatbot->>CloudStorage: Organize and upload media by contact variables
CloudStorage-->>Chatbot: Storage confirmation
Chatbot->>Teacher: Sends submission confirmation
Poem
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
🚀 Deployed on https://deploy-preview-305--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: 0
🧹 Nitpick comments (20)
docs/Use Cases/Use of Google Sheets To Update Contact Variables.md (20)
9-9
: [Style] Revise redundant phrasing in data analysis use case header.
The phrase “at any given point in time” in this sentence is a bit verbose. Consider shortening it to “at any time” for conciseness.🧰 Tools
🪛 LanguageTool
[style] ~9-~9: This phrase is redundant. Consider writing “point” or “time”.
Context: ...class is the teacher doing at any given point in time and overall submission rates. Owing to ...(MOMENT_IN_TIME)
15-15
: [Grammar] Insert missing comma for clarity in the introductory sentence.
In the sentence “Each partner org associated with the SArC, conducting SArC programs with their respective teachers is given a keyword…”, a comma after “teachers” would improve readability.
23-23
: [Grammar] Use the correct article for smoother language.
Change “Provide a art class plan (PDF file)” to “Provide an art class plan (PDF file)” to correctly use the article before a vowel sound.🧰 Tools
🪛 LanguageTool
[misspelling] ~23-~23: Use “an” instead of ‘a’ if the following word starts with a vowel sound, e.g. ‘an article’, ‘an hour’.
Context: ...following key considerations 1. Provide a art class plan (PDF file) in the prefer...(EN_A_VS_AN)
26-26
: [Grammar] Add missing article and comma for clarity.
In “the teacher should not have to make decision on which plan number she has to get this week for any given class,” insert “a” before “decision” and consider adding a comma before “that is” for clarity.🧰 Tools
🪛 LanguageTool
[uncategorized] ~26-~26: Possible missing comma found.
Context: ...o the teacher in sequential order, that is the teacher should not have to make dec...(AI_HYDRA_LEO_MISSING_COMMA)
[uncategorized] ~26-~26: Possible missing article found.
Context: ... is the teacher should not have to make decision on which plan number she has to get thi...(AI_HYDRA_LEO_MISSING_A)
27-27
: [Style] Simplify wordy expression.
The phrase “Owing to the fact that” is unnecessarily wordy. Consider replacing it with “because.”🧰 Tools
🪛 LanguageTool
[style] ~27-~27: ‘Owing to the fact that’ might be wordy. Consider a shorter alternative.
Context: ...le art class plans from the same grade. Owing to the fact that in a week a teacher can go to multiple...(EN_WORDINESS_PREMIUM_OWING_TO_THE_FACT_THAT)
37-37
: [Style] Replace verbose formulations with more direct language.
There are a couple of instances of “so as to” in this section. For example:- so as to serve the next art class plan for that given grade whenever the teacher selects that particular grade. + to serve the next art class plan for that grade whenever the teacher selects it.and
- and increment it logically so as to deliver the plans in a sequential manner. + and increment it logically to deliver the plans sequentially.🧰 Tools
🪛 LanguageTool
[style] ~37-~37: ‘So as to’ expresses purpose and is used in formal texts. Consider using “to”.
Context: ...nters of art class plan for each grade, so as to serve the next art class plan for that ...(SO_AS_TO)
[style] ~37-~37: ‘So as to’ expresses purpose and is used in formal texts. Consider using “to”.
Context: ...re this info and increment it logically so as to deliver the plans in a sequential manne...(SO_AS_TO)
[style] ~37-~37: Consider replacing this phrase with the adverb “sequentially” to avoid wordiness.
Context: ...it logically so as to deliver the plans in a sequential manner. 2. Flags for art class plans for eac...(IN_A_X_MANNER)
38-38
: [Style] Use a more concise phrase.
Instead of “a teacher is able to access only one art class plan at a time,” consider “a teacher can access only one art class plan at a time.”
41-41
: [Grammar] Correct typographical and capitalization errors.
Replace “pulled form the google sheet” with “pulled from the Google Sheet” to fix the typo and capitalize “Google Sheet” appropriately.- pulled form the google sheet + pulled from the Google Sheet🧰 Tools
🪛 LanguageTool
[grammar] ~41-~41: The correct preposition here is “on” or “about”.
Context: ...tact variables to store the information of the grade and art class plan number bei...(INFORMATION_OF)
[uncategorized] ~41-~41: “form” (shape/structure, to make) seems less likely than “from” (“originating from”).
Context: ...r the art class plan after being pulled form the google sheet. 4. Contact variable...(AI_HYDRA_LEO_CP_FORM_FROM)
[grammar] ~41-~41: “Google” is a proper noun and needs to be capitalized.
Context: ... class plan after being pulled form the google sheet. 4. Contact variable to store t...(A_GOOGLE)
43-43
: [Style] Streamline phrasing for clarity.
Consider simplifying “in order to record the completion” to “to record the completion” and add “the” before “grade” for precision.- Contact variable to store the information of grade and art class plan number being submitted in order to record the completion. + Contact variable to store the information of the grade and art class plan number being submitted to record the completion.🧰 Tools
🪛 LanguageTool
[grammar] ~43-~43: The correct preposition here is “on” or “about”.
Context: ...ntact variable to store the information of grade and art class plan number being s...(INFORMATION_OF)
[style] ~43-~43: Consider a shorter alternative to avoid wordiness.
Context: ...d art class plan number being submitted in order to record the completion. This contact hol...(IN_ORDER_TO_PREMIUM)
[grammar] ~43-~43: The correct preposition here is “on” or “about”.
Context: ...ion. This contact holds the information of the previous art class plan that the te...(INFORMATION_OF)
49-49
: [Style] Improve wording for accidental actions.
In the sentence about typos, replace “by mistake” with “accidentally” and ensure proper comma usage after “ex:”.- ...and the teacher by mistake enters cse or caw... + ...and the teacher accidentally enters cse or caw...
53-53
: [Grammar] Insert missing article for correct sentence structure.
Change “sends keyword which is not linked to any active flow” to “sends a keyword which is not linked to any active flow.”🧰 Tools
🪛 LanguageTool
[uncategorized] ~53-~53: Possible missing article found.
Context: ...tact is not part of any flow, and sends keyword which is not linked to any active flow....(AI_HYDRA_LEO_MISSING_A)
21-21
: [Markdown] Remove trailing punctuation from heading.
The heading “### Use case 2: Providing art class plans from a repository of PDFs.” has a trailing period. For consistency with other headings, remove the period.- ### Use case 2: Providing art class plans from a repository of PDFs. + ### Use case 2: Providing art class plans from a repository of PDFs🧰 Tools
🪛 markdownlint-cli2 (0.17.2)
21-21: Trailing punctuation in heading
Punctuation: '.'(MD026, no-trailing-punctuation)
57-57
: [Markdown/Style] Remove trailing punctuation and fix capitalization in heading.
In the heading “### Use case 4: Storing the media submitted by teachers in structured folders in the google cloud storage bucket.”, remove the trailing period and change “google cloud” to “Google Cloud” for proper noun capitalization.- ### Use case 4: Storing the media submitted by teachers in structured folders in the google cloud storage bucket. + ### Use case 4: Storing the media submitted by teachers in structured folders in Google Cloud storage bucket🧰 Tools
🪛 LanguageTool
[grammar] ~57-~57: “Google” is a proper noun and needs to be capitalized.
Context: ...y teachers in structured folders in the google cloud storage bucket. As SArC engages ...(A_GOOGLE)
🪛 markdownlint-cli2 (0.17.2)
57-57: Trailing punctuation in heading
Punctuation: '.'(MD026, no-trailing-punctuation)
58-58
: [Style] Refine sentence structure and correct minor grammatical issues.
Rephrase “As SArC engages a numerous teachers, across numerous grades and has multiple partners (30+)” to remove the unnecessary article and comma, and adjust subsequent prepositions (e.g. “in terms on” should become “in terms of”).- As SArC engages a numerous teachers, across numerous grades and has multiple partners (30+). + As SArC engages numerous teachers across various grades and has multiple partners (30+).Also, ensure that any occurrence of “in terms on actual engagement” is updated to “in terms of actual engagement.”
🧰 Tools
🪛 LanguageTool
[uncategorized] ~58-~58: Did you mean “numerous teachers”?
Context: ... cloud storage bucket. As SArC engages a numerous teachers, across numerous grades and has multipl...(A_NUMEROUS_WORDS)
[uncategorized] ~58-~58: Possible missing comma found.
Context: ... communication material. And at another level this also helps to triangulate the prog...(AI_HYDRA_LEO_MISSING_COMMA)
[uncategorized] ~58-~58: The preposition ‘of’ seems more likely in this position.
Context: ...chers from any particular org, in terms on actual engagement and completion of the...(AI_HYDRA_LEO_REPLACE_ON_OF)
60-60
: [Grammar] Correct proper noun usage and punctuation.
Change “google drive” to “Google Drive” and add the necessary comma if needed.- dumped into the “uploads” folder of the google drive. + dumped into the “uploads” folder of the Google Drive.🧰 Tools
🪛 LanguageTool
[uncategorized] ~60-~60: Possible missing comma found.
Context: ...ed on the values present in the contact variables as opposed to being dumped into the “up...(AI_HYDRA_LEO_MISSING_COMMA)
[grammar] ~60-~60: Did you mean “Google Drive”?
Context: ...dumped into the “uploads” folder of the google drive. An example folder structure. "Upload...(GOOGLE_PRODUCTS)
66-66
: [Style] Fix idiomatic expression.
Replace “from the get go” with “from the get-go” for correct spelling and style.- from the get go + from the get-go🧰 Tools
🪛 LanguageTool
[uncategorized] ~66-~66: In this idiom, “get-go” is spelled with a hyphen.
Context: ...ed by a teacher will be sorted from the get go into this folder structure. ## Credi...(FROM_THE_GET_GO)
70-70
: [Style] Enhance clarity with appropriate punctuation in credits.
Consider adding a comma before “and” in the list to improve readability (e.g., “Rukhmini Bhatia, Tanushree Patel, and Surabhi Shinde”).- Shoutout to Rukhmini Bhatia, Tanushree Patel and Surabhi Shinde from the Saturday Art Class team... + Shoutout to Rukhmini Bhatia, Tanushree Patel, and Surabhi Shinde from the Saturday Art Class team...🧰 Tools
🪛 LanguageTool
[uncategorized] ~70-~70: Possible missing comma found.
Context: ...previous iteration of implementing this program which ran successfully over the chatbot...(AI_HYDRA_LEO_MISSING_COMMA)
73-73
: [Grammar] Correct verb form for clarity.
Change “involved implement a fairly complex logic” to “involved implementing a fairly complex logic.”- This project involved implement a fairly complex logic + This project involved implementing a fairly complex logic🧰 Tools
🪛 LanguageTool
[uncategorized] ~73-~73: Possible missing preposition found.
Context: ... ## Conclusion This project involved implement a fairly complex logic that informs the...(AI_HYDRA_LEO_MISSING_TO)
75-75
: [Grammar] Insert missing preposition in the call to action.
Modify “refer this documentation” to “refer to this documentation” for correct preposition usage.- refer this documentation. + refer to this documentation.🧰 Tools
🪛 LanguageTool
[uncategorized] ~75-~75: Possible missing preposition found.
Context: ... chatbot flows more logic driven, refer this documentation. Reach out to tejas@gli...(AI_HYDRA_LEO_MISSING_TO)
77-77
: [Markdown] Convert bare email address to a mailto link.
For consistency and enhanced usability, change “Reach out to [email protected]” to a proper mailto link.- Reach out to [email protected] to chat further on any details shared here. + Reach out to [[email protected]](mailto:[email protected]) to chat further on any details shared here.🧰 Tools
🪛 markdownlint-cli2 (0.17.2)
77-77: Bare URL used
null(MD034, no-bare-urls)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
docs/Use Cases/Use of Google Sheets To Update Contact Variables.md
(1 hunks)
🧰 Additional context used
🪛 LanguageTool
docs/Use Cases/Use of Google Sheets To Update Contact Variables.md
[style] ~9-~9: This phrase is redundant. Consider writing “point” or “time”.
Context: ...class is the teacher doing at any given point in time and overall submission rates. Owing to ...
(MOMENT_IN_TIME)
[uncategorized] ~16-~16: Possible missing comma found.
Context: ...ing SArC programs with their respective teachers is given a keyword. This keyword is use...
(AI_HYDRA_LEO_MISSING_COMMA)
[misspelling] ~23-~23: Use “an” instead of ‘a’ if the following word starts with a vowel sound, e.g. ‘an article’, ‘an hour’.
Context: ...following key considerations 1. Provide a art class plan (PDF file) in the prefer...
(EN_A_VS_AN)
[uncategorized] ~26-~26: Possible missing comma found.
Context: ...o the teacher in sequential order, that is the teacher should not have to make dec...
(AI_HYDRA_LEO_MISSING_COMMA)
[uncategorized] ~26-~26: Possible missing article found.
Context: ... is the teacher should not have to make decision on which plan number she has to get thi...
(AI_HYDRA_LEO_MISSING_A)
[style] ~27-~27: ‘Owing to the fact that’ might be wordy. Consider a shorter alternative.
Context: ...le art class plans from the same grade. Owing to the fact that in a week a teacher can go to multiple...
(EN_WORDINESS_PREMIUM_OWING_TO_THE_FACT_THAT)
[style] ~37-~37: ‘So as to’ expresses purpose and is used in formal texts. Consider using “to”.
Context: ...nters of art class plan for each grade, so as to serve the next art class plan for that ...
(SO_AS_TO)
[style] ~37-~37: ‘So as to’ expresses purpose and is used in formal texts. Consider using “to”.
Context: ...re this info and increment it logically so as to deliver the plans in a sequential manne...
(SO_AS_TO)
[style] ~37-~37: Consider replacing this phrase with the adverb “sequentially” to avoid wordiness.
Context: ...it logically so as to deliver the plans in a sequential manner. 2. Flags for art class plans for eac...
(IN_A_X_MANNER)
[style] ~39-~39: As a shorter alternative for ‘able to’, consider using “can”.
Context: ...or each grade, to ensure that a teacher is able to access only one art class plan at a tim...
(BE_ABLE_TO)
[grammar] ~41-~41: The correct preposition here is “on” or “about”.
Context: ...tact variables to store the information of the grade and art class plan number bei...
(INFORMATION_OF)
[uncategorized] ~41-~41: “form” (shape/structure, to make) seems less likely than “from” (“originating from”).
Context: ...r the art class plan after being pulled form the google sheet. 4. Contact variable...
(AI_HYDRA_LEO_CP_FORM_FROM)
[grammar] ~41-~41: “Google” is a proper noun and needs to be capitalized.
Context: ... class plan after being pulled form the google sheet. 4. Contact variable to store t...
(A_GOOGLE)
[grammar] ~43-~43: The correct preposition here is “on” or “about”.
Context: ...ntact variable to store the information of grade and art class plan number being s...
(INFORMATION_OF)
[style] ~43-~43: Consider a shorter alternative to avoid wordiness.
Context: ...d art class plan number being submitted in order to record the completion. This contact hol...
(IN_ORDER_TO_PREMIUM)
[grammar] ~43-~43: The correct preposition here is “on” or “about”.
Context: ...ion. This contact holds the information of the previous art class plan that the te...
(INFORMATION_OF)
[uncategorized] ~48-~48: Possible missing comma found.
Context: ...r sometimes makes typos in entering the keyword ex: the expected keyword is cae, and th...
(AI_HYDRA_LEO_MISSING_COMMA)
[style] ~48-~48: ‘by mistake’ might be wordy. Consider a shorter alternative.
Context: ...xpected keyword is cae, and the teacher by mistake enters cse or caw or something similar....
(EN_WORDINESS_PREMIUM_BY_MISTAKE)
[uncategorized] ~53-~53: Possible missing article found.
Context: ...tact is not part of any flow, and sends keyword which is not linked to any active flow....
(AI_HYDRA_LEO_MISSING_A)
[grammar] ~57-~57: “Google” is a proper noun and needs to be capitalized.
Context: ...y teachers in structured folders in the google cloud storage bucket. As SArC engages ...
(A_GOOGLE)
[uncategorized] ~58-~58: Did you mean “numerous teachers”?
Context: ... cloud storage bucket. As SArC engages a numerous teachers, across numerous grades and has multipl...
(A_NUMEROUS_WORDS)
[uncategorized] ~58-~58: Possible missing comma found.
Context: ... communication material. And at another level this also helps to triangulate the prog...
(AI_HYDRA_LEO_MISSING_COMMA)
[uncategorized] ~58-~58: The preposition ‘of’ seems more likely in this position.
Context: ...chers from any particular org, in terms on actual engagement and completion of the...
(AI_HYDRA_LEO_REPLACE_ON_OF)
[uncategorized] ~60-~60: Possible missing comma found.
Context: ...ed on the values present in the contact variables as opposed to being dumped into the “up...
(AI_HYDRA_LEO_MISSING_COMMA)
[grammar] ~60-~60: Did you mean “Google Drive”?
Context: ...dumped into the “uploads” folder of the google drive. An example folder structure. "Upload...
(GOOGLE_PRODUCTS)
[uncategorized] ~65-~65: Possible missing comma found.
Context: ...d684bc-53c5-4b7f-9568-321d3e4ecaeb" /> Similarly based on the contact variables of org n...
(AI_HYDRA_LEO_MISSING_COMMA)
[uncategorized] ~66-~66: In this idiom, “get-go” is spelled with a hyphen.
Context: ...ed by a teacher will be sorted from the get go into this folder structure. ## Credi...
(FROM_THE_GET_GO)
[uncategorized] ~70-~70: Possible missing comma found.
Context: ...previous iteration of implementing this program which ran successfully over the chatbot...
(AI_HYDRA_LEO_MISSING_COMMA)
[uncategorized] ~73-~73: Possible missing preposition found.
Context: ... ## Conclusion This project involved implement a fairly complex logic that informs the...
(AI_HYDRA_LEO_MISSING_TO)
[uncategorized] ~75-~75: Possible missing preposition found.
Context: ... chatbot flows more logic driven, refer this documentation. Reach out to tejas@gli...
(AI_HYDRA_LEO_MISSING_TO)
🪛 markdownlint-cli2 (0.17.2)
docs/Use Cases/Use of Google Sheets To Update Contact Variables.md
21-21: Trailing punctuation in heading
Punctuation: '.'
(MD026, no-trailing-punctuation)
57-57: Trailing punctuation in heading
Punctuation: '.'
(MD026, no-trailing-punctuation)
77-77: Bare URL used
null
(MD034, no-bare-urls)
Uploaded the summary of work for done for SArC which lists 4 use cases of using google sheets and contact variables and custom expressions.
Summary by CodeRabbit