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

add snapshot page detailing legacy syntax #6803

Open
wants to merge 11 commits into
base: current
Choose a base branch
from

Conversation

mirnawong1
Copy link
Contributor

@mirnawong1 mirnawong1 commented Jan 21, 2025

this pr adds a new legacy snapshot syntax page that details how to configure snapshots using the now legacy jinja config block way.

since there might be instnces where configuring snapshots this way is beneficial, we're adding a legacy page, this ensures users still know how to configure snaphots this way and it doesn't get lost.

  • wait for Joel to confirm tech accuracy

related to #6561

docs notion


🚀 Deployment available! Here are the direct links to the updated files:

@mirnawong1 mirnawong1 requested a review from a team as a code owner January 21, 2025 15:56
Copy link

vercel bot commented Jan 21, 2025

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

Name Status Preview Updated (UTC)
docs-getdbt-com ✅ Ready (Inspect) Visit Preview Feb 7, 2025 9:29am

@github-actions github-actions bot added content Improvements or additions to content size: large This change will more than a week to address and might require more than one person Docs team Authored by the Docs team @dbt Labs labels Jan 21, 2025
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

vale

website/docs/docs/build/snapshots.md|79 col 4| [custom.Typos] Oops there's a typo -- did you really mean 'updated_at'?
website/docs/docs/build/snapshots.md|79 col 168| [custom.Typos] Oops there's a typo -- did you really mean 'updated_at'?
website/docs/docs/build/snapshots.md|80 col 4| [custom.Typos] Oops there's a typo -- did you really mean 'invalidate_hard_deletes'?
website/docs/docs/build/snapshots.md|82 col 56| [custom.Typos] Oops there's a typo -- did you really mean 'e.g.'?
website/docs/docs/build/snapshots.md|82 col 56| [custom.LatinAbbreviations] Avoid Latin abbreviations: 'for example'. Consider using 'e.g' instead.
website/docs/docs/build/snapshots.md|90 col 78| [custom.LatinAbbreviations] Avoid Latin abbreviations: 'for example'. Consider using 'e.g' instead.
website/docs/docs/build/snapshots.md|90 col 78| [custom.Typos] Oops there's a typo -- did you really mean 'e.g.'?
website/docs/docs/build/snapshots.md|252 col 24| [custom.Typos] Oops there's a typo -- did you really mean 'v1.9'?
website/docs/docs/build/snapshots.md|253 col 6| [custom.Typos] Oops there's a typo -- did you really mean 'v1.9'?

website/docs/docs/build/snapshots.md Show resolved Hide resolved
website/docs/docs/build/snapshots.md Outdated Show resolved Hide resolved
website/docs/docs/build/snapshots.md Outdated Show resolved Hide resolved
website/docs/docs/build/snapshots.md Show resolved Hide resolved
website/docs/docs/build/snapshots.md Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content Improvements or additions to content Docs team Authored by the Docs team @dbt Labs size: large This change will more than a week to address and might require more than one person
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant