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

made changes to category (split) #2415

Merged
merged 4 commits into from
Jan 30, 2025
Merged

made changes to category (split) #2415

merged 4 commits into from
Jan 30, 2025

Conversation

GabyRangelB
Copy link
Contributor

@GabyRangelB GabyRangelB commented Jan 30, 2025

Pull Request

Summary & Motivation

Self-review

If this is a same-day request, please flag that in our Slack channel!

  • Update due date, assignee, and priority on our
    TEAMster Asana Project
  • Format has been run on all modified files

SQL

  • Ensure you are using the union_dataset_join_clause() macro for queries that employ any
    models using these datasets: deanslist edplan iready overgrad pearson powerschool
    renlearn titan

  • All distinct usage must be accompanied by an comment explaining it's necessity

  • If you are adding a new external source, run:

    dbt run-operation stage_external_sources --vars
    "ext_full_refresh: true"
    

Troubleshooting


@GabyRangelB GabyRangelB requested a review from a team as a code owner January 30, 2025 13:27
@anthonygwalters anthonygwalters merged commit 97022d4 into main Jan 30, 2025
6 checks passed
@anthonygwalters anthonygwalters deleted the missing_vals branch January 30, 2025 15:30
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