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

Fix docs so it does not reference non-existing get_dbt_dataset #1034

Merged
merged 3 commits into from
Jun 7, 2024

Conversation

tatiana
Copy link
Collaborator

@tatiana tatiana commented Jun 7, 2024

The documentation was outdated.

The method get_dbt_dataset no longer exists. It used to exist in older versions of Cosmos (before 1.1) when the URIs respected the format:
Dataset(f"DBT://{connection_id.upper()}/{project_name.upper()}/{model_name.upper()}")

More information on why we changed this: #305

Closes: #1032

@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Jun 7, 2024
Copy link

netlify bot commented Jun 7, 2024

Deploy Preview for sunny-pastelito-5ecb04 ready!

Name Link
🔨 Latest commit 4bfd2f0
🔍 Latest deploy log https://app.netlify.com/sites/sunny-pastelito-5ecb04/deploys/666308e87920df000887ca93
😎 Deploy Preview https://deploy-preview-1034--sunny-pastelito-5ecb04.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@dosubot dosubot bot added the area:docs Relating to documentation, changes, fixes, improvement label Jun 7, 2024
docs/configuration/scheduling.rst Outdated Show resolved Hide resolved
docs/configuration/scheduling.rst Outdated Show resolved Hide resolved
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jun 7, 2024
@tatiana tatiana merged commit c47e104 into main Jun 7, 2024
61 checks passed
@tatiana tatiana deleted the update-scheduling-docs branch June 7, 2024 13:25
pankajkoti pushed a commit that referenced this pull request Jun 7, 2024
[The
documentation](https://astronomer.github.io/astronomer-cosmos/configuration/scheduling.html)
was outdated.

The method `get_dbt_dataset` no longer exists. It used to exist in older
versions of Cosmos (before 1.1) when the URIs respected the format:

`Dataset(f"DBT://{connection_id.upper()}/{project_name.upper()}/{model_name.upper()}")`

More information on why we changed this:
#305

Closes: #1032
(cherry picked from commit c47e104)
@pankajkoti pankajkoti mentioned this pull request Jun 7, 2024
pankajkoti added a commit that referenced this pull request Jun 7, 2024
Bug fixes

* Bring back ``dataset`` as a required field for BigQuery profile by
@pankajkoti in #1033

Enhancements

* Only run ``dbt deps`` when there are dependencies by @tatiana in #1030

Docs

* Fix docs so it does not reference non-existing ``get_dbt_dataset`` by
@tatiana in #1034

---------

Co-authored-by: Tatiana Al-Chueyr <tatiana.alchueyr@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:docs Relating to documentation, changes, fixes, improvement lgtm This PR has been approved by a maintainer size:S This PR changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Docs]: Update outdated Data-Aware Scheduling docs
2 participants