-
Notifications
You must be signed in to change notification settings - Fork 5k
Fix(Source Google Drive): Date out of range (Do Not Merge) #69340
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: master
Are you sure you want to change the base?
Fix(Source Google Drive): Date out of range (Do Not Merge) #69340
Conversation
👋 Greetings, Airbyte Team Member!Here are some helpful tips and reminders for your convenience. Helpful Resources
PR Slash CommandsAirbyte Maintainers (that's you!) can execute the following slash commands on your PR:
|
| ::: | ||
|
|
||
| :::warning | ||
| Files that live in the Google Drive trash are still returned by the Drive API when the folder is scanned. The connector will therefore sync trashed files—including Excel workbooks and other file types—whenever they match your path patterns. Permanently delete or restore unwanted files before running a sync if you do not want them replicated. |
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.
[Google.Will] Avoid using 'will'.
|
|
Deploy preview for airbyte-docs ready! ✅ Preview Built with commit 8289136. |
…sh mismatch Co-Authored-By: unknown <>
What
OC Issue: https://github.com/airbytehq/oncall/issues/10097
Currently, the exce reader is getting:
ValueError('year 20225 is out of range')”How
This CDK Change: airbytehq/airbyte-python-cdk#850 adds a Openpyxl as a fallback option for parsing Excel files. We are making a dev version in this PR and testing on a cloud environment after getting good results locally.
Review guide
User Impact
Can this PR be safely reverted and rolled back?