Skip to content

Minor syntax update for date casting for dbt 1.6.3 support#936

Closed
utsavpaudel wants to merge 1 commit into
tuva-health:mainfrom
utsavpaudel:v1.6.3_fix
Closed

Minor syntax update for date casting for dbt 1.6.3 support#936
utsavpaudel wants to merge 1 commit into
tuva-health:mainfrom
utsavpaudel:v1.6.3_fix

Conversation

@utsavpaudel

Copy link
Copy Markdown
Contributor

Describe your changes

Updated date casting logic to more adapted format in Tuva package.
This enables Github Action run error free in demo repo and solves tuva-health/demo#53

How has this been tested?

Ran dbt build -s asc__generate_encounter_id in snowflake, bigquery and redshift

Reviewer focus

Please summarize the specific items you’d like the reviewer(s) to look into.

Checklist before requesting a review

  • I have added at least one Github label to this PR (bug, enhancement, breaking change,...)
  • My code follows style guidelines
  • (New models) YAML files are categorized by sub folder and models listed in alphabetical order
  • (New models) I have added a config to each new model to enable it for claims and/or clinical data
  • (New models) I have added the variable tuva_last_run to the final output
  • (Optional) I have recorded a Loom to explain this PR

Package release checklist

  • I have updated dbt docs
  • I have updated the version number in the dbt_project.yml

(Optional) Gif of how this PR makes you feel

Loom link

@utsavpaudel

Copy link
Copy Markdown
Contributor Author

This PR intended to solve tuva-health/demo#53 issue, which tuva-health/demo#55 already does. Since the second one is merged, this PR becomes redundant - hence closing this PR.

@utsavpaudel utsavpaudel closed this Jul 4, 2025
@github-project-automation github-project-automation Bot moved this from 👀 Ready for Review to ✅ Done in The Tuva Project Jul 4, 2025
@utsavpaudel
utsavpaudel deleted the v1.6.3_fix branch July 4, 2025 08:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

1 participant