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

Update Jaffle Shop tutorial #165

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

thomasyu888
Copy link

These are minor changes - so feel free to close if the images aren't up to spec.

  • The dbt jaffle shop tutorial has updated, so changing from amount / 100 to amount turns into schema change (which was cool to see, but the diff wasn't as cool)
  • The UI has changed so I updated those. Did you want me to add the "Approve checklist image"?

Side note: great link about cairo causing the build to fail. this did fix it export DYLD_FALLBACK_LIBRARY_PATH=/opt/homebrew/lib

Heres a screenshot of the built image.
image

@kentwelcome
Copy link
Member

Thanks for helping to update the images.
I’m curious about the reason for changing amount / 100 as amount to amount / 20 as amount. Does this mean it better represents a real change in the definition of amount, rather than just appending 00 as in our original example?

@thomasyu888
Copy link
Author

thomasyu888 commented Mar 19, 2025

  • The dbt jaffle shop tutorial has updated, so changing from amount / 100 to amount turns into schema change (which was cool to see, but the diff wasn't as cool)

@kentwelcome The dbt jaffle shop tutorial seems to have been updated, so changing from amount / 100 to amount turns into schema change (which was cool to see, but the diff wasn't as cool - i liked the checker boarded white/red). I tried to cast(amount as integer) as amount, but that didn't seem to do the trick (a dbt novice here)

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants