-
Notifications
You must be signed in to change notification settings - Fork 382
Revamp DBT page #4479
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
base: main
Are you sure you want to change the base?
Revamp DBT page #4479
Conversation
The latest updates on your projects. Learn more about Vercel for GitHub.
3 Skipped Deployments
|
docs/integrations/data-ingestion/etl-tools/dbt/features-and-configurations.md
Outdated
Show resolved
Hide resolved
There are many places where we reference the connector as an adapter/plugin. Can we align the terminology to use only adapter? |
@koletzilla: do you mind if I push commits directly to your branch to do some more reorg? It's probably easier for both of us. I can take care of some of the styling issues Bentsi mentioned, too. Thanks for working on this, it's a massive improvement! |
Co-authored-by: Bentsi Leviav <[email protected]>
Co-authored-by: Bentsi Leviav <[email protected]>
That was bothering me as well 😅 Changed to |
Hi! Yes please! Feel free to add it to the branch. I already took care of Bentsi's, but feel free to change other things. Just as a note: I'd rather merge this PR asap so we can start adding smaller, more granular changes to the new docs. If the changes are small, do you mind waiting until this PR is merged? If the changes are big/moves a lot of text, using this PR feels good to me 😄 |
Summary
Related to ClickHouse/dbt-clickhouse#484
Changes required to make the CH dbt's docs the single source of truth for the dbt-clickhouse plugin.
Checklist