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

Psatnmsqt #2384

Merged
merged 6 commits into from
Jan 17, 2025
Merged

Psatnmsqt #2384

merged 6 commits into from
Jan 17, 2025

Conversation

GabyRangelB
Copy link
Contributor

@GabyRangelB GabyRangelB commented Jan 17, 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 review from a team as code owners January 17, 2025 21:53
@cbini cbini merged commit a4e7cca into main Jan 17, 2025
6 checks passed
@cbini cbini deleted the psatnmsqt branch January 17, 2025 22:36
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