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

[daggy-u] [dbt] - Add Lesson 5 (DEV-57) #19947

Merged
merged 14 commits into from
Feb 27, 2024
Merged

Conversation

erinkcochran87
Copy link
Contributor

@erinkcochran87 erinkcochran87 commented Feb 21, 2024

Summary & Motivation

This PR adds Lesson 5 of the new dbt module to Dagster University.

TODOs

  • Add screenshots
  • Update code snippets to use file import

How I Tested These Changes

@erinkcochran87 erinkcochran87 added the area: dagster-university Related to Dagster University label Feb 21, 2024
Copy link

Deploy preview for dagster-university ready!

✅ Preview
https://dagster-university-ox7ql9g00-elementl.vercel.app
https://erin-dev-57-lesson-5.dagster-university.dagster-docs.io

Built with commit 90d4238.
This pull request is being automatically deployed with vercel-action

@tacastillo tacastillo self-requested a review February 26, 2024 22:20
@tacastillo tacastillo marked this pull request as ready for review February 26, 2024 22:21
Copy link
Contributor

@tacastillo tacastillo left a comment

Choose a reason for hiding this comment

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

Left some suggestions you can batch apply, if you'd like!


# Overview

In Lesson 3, you loaded your dbt project's models as assets into Dagster. You also materialized some of those models.
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we also need a reference to Lesson 4 here?

@erinkcochran87 erinkcochran87 merged commit 822b4a4 into master Feb 27, 2024
1 check was pending
@erinkcochran87 erinkcochran87 deleted the erin/dev-57-lesson-5 branch February 27, 2024 18:44
PedramNavid pushed a commit that referenced this pull request Feb 28, 2024
## Summary & Motivation

This PR adds Lesson 5 of the new dbt module to Dagster University.

TODOs

- [x] Add screenshots
- [ ] Update code snippets to use file import

## How I Tested These Changes

---------

Co-authored-by: Tim Castillo <[email protected]>
cmpadden pushed a commit that referenced this pull request Feb 28, 2024
## Summary & Motivation

This PR adds Lesson 5 of the new dbt module to Dagster University.

TODOs

- [x] Add screenshots
- [ ] Update code snippets to use file import

## How I Tested These Changes

---------

Co-authored-by: Tim Castillo <[email protected]>
alangenfeld pushed a commit that referenced this pull request Feb 28, 2024
This PR adds Lesson 5 of the new dbt module to Dagster University.

TODOs

- [x] Add screenshots
- [ ] Update code snippets to use file import

---------

Co-authored-by: Tim Castillo <[email protected]>
PedramNavid pushed a commit that referenced this pull request Mar 28, 2024
## Summary & Motivation

This PR adds Lesson 5 of the new dbt module to Dagster University.

TODOs

- [x] Add screenshots
- [ ] Update code snippets to use file import

## How I Tested These Changes

---------

Co-authored-by: Tim Castillo <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: dagster-university Related to Dagster University
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants