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
🐛 Source Shopify: Fixed bug when start date
was not provided, but the stream was using it
#35952
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
airbyte-integrations/connectors/source-shopify/source_shopify/streams/base_streams.py
Outdated
Show resolved
Hide resolved
Do we need to add |
# but if the customer has already switched to the new version, | ||
# corrected the old config and switches back to the new version, | ||
# we should try to migrate the modified old custom reports. | ||
none_values: List[str] = [None, "None"] |
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.
Could this be a tuple? Sorry
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.
Sure, but what is wrong having the List
here?
What
Resolving:
How
start date
value andfallback
to thedefault_start_date
value of2018-01-01
, if thestart_date
was not provided in theconfig
config migration
to gracefully migrate the configs with no / badstart_date
valuesunit_test