-
Notifications
You must be signed in to change notification settings - Fork 976
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
An example of separating semantic model (pure yaml) from dbt model (yaml, sql, python) #6305
Conversation
…l (yaml, sql, python)
Hello!👋 Thanks for contributing to the dbt product documentation and opening this pull request! ✨ |
@albertlieyingadrian is attempting to deploy a commit to the dbt-labs Team on Vercel. A member of the Team first needs to authorize it. |
Or actually, adding a link to the tutorial of setting it up on Snowflake might be sufficient https://docs.getdbt.com/guides/sl-snowflake-qs?step=9 |
Lmk what else I need to change @runleonarun |
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.
hey @albertlieyingadrian , this is such a great pr! really beautifully written and thank you so much for opening this up to clarify! I'll probably follow up so i can link to your new header additions from the 'semantic structure' page and the quickstart, like you suggested.
i tweaked it a little and turn the h3 headers into expandable toggles, which are also linkable, just to shorten the scrolling needed and ensure the explanation is viewed too.
I'm happy to merge this for you if you have no other comments/feedback. thank you again, this is such a great PR! ✨
No other comments. Please merge @mirnawong1 |
@albertlieyingadrian Woah this was you? - I was impressed on how the documentation had improved over the course of a week! |
What are you changing in this pull request and why?
It isn't a clear-cut to find an example of
sub-folder
semantic approach. Hence, adding an example of it. Additionally, adding it to jaffle-shop repo would be wonderful.Resolves #6306
Checklist