Skip to content
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

Update the warning on stripe full-refresh-overwrite #21007

Merged
merged 2 commits into from
Mar 17, 2023

Conversation

killthekitten
Copy link
Contributor

@killthekitten killthekitten commented Jan 4, 2023

What & how

This commit fixes the warning message in the stripe source doc. The previous message wasn't specific enough and could be interpreted as if all the data older than 30 days will be deleted. This has already created some confusion while our team was debugging a different issue.

Related to #19515.

This commit fixes the warning message in the stripe source doc. The previous message wasn't specific enough and could be interpreted as if all the data older than 30 days will be deleted.
@CLAassistant
Copy link

CLAassistant commented Jan 4, 2023

CLA assistant check
All committers have signed the CLA.

@octavia-squidington-iv octavia-squidington-iv added area/documentation Improvements or additions to documentation community labels Jan 4, 2023
@marcosmarxm marcosmarxm merged commit 3b0a710 into airbytehq:master Mar 17, 2023
erohmensing pushed a commit that referenced this pull request Mar 22, 2023
This commit fixes the warning message in the stripe source doc. The previous message wasn't specific enough and could be interpreted as if all the data older than 30 days will be deleted.

Co-authored-by: Marcos Marx <marcosmarxm@users.noreply.github.com>
erohmensing pushed a commit that referenced this pull request Mar 22, 2023
This commit fixes the warning message in the stripe source doc. The previous message wasn't specific enough and could be interpreted as if all the data older than 30 days will be deleted.

Co-authored-by: Marcos Marx <marcosmarxm@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation Improvements or additions to documentation community
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants