Skip to content

Conversation

@shijithkjayan
Copy link
Member

@shijithkjayan shijithkjayan commented Oct 9, 2025

Clarify that dates must be in DD/MM/YYYY format.

Summary by CodeRabbit

  • Documentation
    • Standardized example date format to DD/MM/YYYY.
    • Added guidance that Google Sheets dates must use DD/MM/YYYY for correct matching, with locale/custom format tips.
    • Introduced a prominent Note highlighting required date format and impact if misformatted.
    • Expanded calendar reference list to include @calendar.current_year.

Clarify that dates must be in DD/MM/YYYY format.
@shijithkjayan shijithkjayan requested a review from madclaws October 9, 2025 11:15
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 9, 2025

Walkthrough

Updated the "Link Google Sheets" flow action documentation: standardized example date format to DD/MM/YYYY, added a Note specifying Google Sheets must store dates in DD/MM/YYYY (with locale/custom-format guidance), and expanded calendar functions to include @calendar.current_year.

Changes

Cohort / File(s) Summary
Docs: Google Sheets linking flow
docs/4. Product Features/03. Flows/2. Flow Actions/12. Link Google Sheets.md
Changed example search key from D/MM/YYYY to DD/MM/YYYY; added explicit Note that Google Sheet dates must be in DD/MM/YYYY and guidance to set locale/use custom date format; added @calendar.current_year to calendar functions list.

Sequence Diagram(s)

(omitted — changes are documentation-only and do not modify control flow)

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested reviewers

  • Fawas003

Poem

A rabbit hops through sheets so bright,
Aligning dates to fit just right.
DD/MM/YYYY in line,
The flows and cells now sing in time.
🐇📅

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title Check ⚠️ Warning The title accurately describes the main change—updating the Google Sheets documentation to clarify the date format—but it includes an emoji, which is considered noise and detracts from a concise, professional title. Remove the “:recycle:” emoji and consider simplifying the prefix, for example: “Update Google Sheets documentation to clarify DD/MM/YYYY date format.”
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch update-date-format-info

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c3f9c92 and d1eeb65.

📒 Files selected for processing (1)
  • docs/4. Product Features/03. Flows/2. Flow Actions/12. Link Google Sheets.md (1 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
  • docs/4. Product Features/03. Flows/2. Flow Actions/12. Link Google Sheets.md

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link

github-actions bot commented Oct 9, 2025

@github-actions github-actions bot temporarily deployed to pull request October 9, 2025 11:16 Inactive
_As per above example, the sheet `Daily Activity` is used to read the content from, and the variable `@calendar.current_date` D/MM/YYYY being passed from Glific to the Google sheet is being used as the search key to identify the relevant row to send the content from_
_As per above example, the sheet `Daily Activity` is used to read the content from, and the variable `@calendar.current_date` DD/MM/YYYY being passed from Glific to the Google sheet is being used as the search key to identify the relevant row to send the content from_

**Note**: It is important to note that dates in the sheet should be in the format `DD/MM/YYYY`. Any other format will not work as expected.
Copy link
Collaborator

@madclaws madclaws Oct 9, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The issue was even though we only send the date in DD/MM/YYYY from Glific, Google sheet might be considering it as MM/DD/YYYY. So we need to make sure either the Settings -> Locale is India or the date column should be in DD/MM/YYYY format, which can be done from Format -> Number -> Custom Date and Time for a column.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed d1eeb65

Add instructions for setting DD/MM/YYYY format via locale or custom
formatting options in Google Sheets.
@shijithkjayan shijithkjayan requested a review from madclaws October 9, 2025 16:45
@github-actions github-actions bot temporarily deployed to pull request October 9, 2025 16:47 Inactive
@shijithkjayan shijithkjayan merged commit 71213cd into main Oct 9, 2025
7 checks passed
@akanshaaa19 akanshaaa19 deleted the update-date-format-info branch October 22, 2025 09:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Reading date from GSheet, messes up the format and misinterprets DD/MM as MM/DD

3 participants