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

Docs: fixing wrong jinja example and stating how airbyte decides last date cursor #34000

Merged
merged 5 commits into from
Jan 9, 2024

Conversation

girarda
Copy link
Contributor

@girarda girarda commented Jan 8, 2024

What

*Documents which date do pick as the state to use in the next sync
*Fixes an example that was using {{stream_interval.start_date}} and {{ stream_interval.end_date }} instead of the correct way {{ stream_interval.start_time }} and {{ stream_interval.end_time }}

This is a non-forked PR from #33847

How

  • Update the doc

Copy link

vercel bot commented Jan 8, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
airbyte-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 8, 2024 2:50pm

@octavia-squidington-iii octavia-squidington-iii added the area/documentation Improvements or additions to documentation label Jan 8, 2024
@girarda girarda marked this pull request as draft January 8, 2024 13:31
@girarda girarda merged commit 62d3b56 into master Jan 9, 2024
22 checks passed
@girarda girarda deleted the alex/update_jinja_doc branch January 9, 2024 01:40
jatinyadav-cc pushed a commit to ollionorg/datapipes-airbyte that referenced this pull request Feb 26, 2024
… date cursor (airbytehq#34000)

Co-authored-by: Segers <gabrielsegers@hotmail.com>
jatinyadav-cc pushed a commit to ollionorg/datapipes-airbyte that referenced this pull request Feb 26, 2024
… date cursor (airbytehq#34000)

Co-authored-by: Segers <gabrielsegers@hotmail.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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants