Skip to content

DM-51936 : Migrate tables for v2 feature development#183

Merged
tcjennings merged 1 commit intotickets/DM-51337/releasefrom
tickets/DM-51396/v2_tables
Jul 7, 2025
Merged

DM-51936 : Migrate tables for v2 feature development#183
tcjennings merged 1 commit intotickets/DM-51337/releasefrom
tickets/DM-51396/v2_tables

Conversation

@tcjennings
Copy link
Copy Markdown
Collaborator

feat(db): Add migrations and models for v2 tables

  • these table migrations are subject to change as v2 feature development continues. Expect to denormalize columns from "metadata" fields and potentially apply validation constraints, indexes, and etc.

@tcjennings tcjennings force-pushed the tickets/DM-51396/v2_tables branch from f011891 to b35ea06 Compare June 23, 2025 19:29
@tcjennings tcjennings marked this pull request as ready for review June 23, 2025 19:31
Base automatically changed from tickets/DM-51397/py312_types to tickets/DM-51337/release June 24, 2025 14:28
@tcjennings tcjennings force-pushed the tickets/DM-51396/v2_tables branch 3 times, most recently from ac2ebfb to 7782fc0 Compare June 25, 2025 16:26
feat(db): Extend v2 model fields with pydantic aliases
@tcjennings tcjennings force-pushed the tickets/DM-51396/v2_tables branch from 7782fc0 to 4ece397 Compare June 30, 2025 16:38
Copy link
Copy Markdown
Member

@ctslater ctslater left a comment

Choose a reason for hiding this comment

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

Looks good. We talked through my main issue which was the Campaign/CampaignModel/CampaignBase structure, and since that is revised in later PRs then ok to go ahead with this one as is.

@tcjennings tcjennings merged commit 259a904 into tickets/DM-51337/release Jul 7, 2025
5 of 7 checks passed
@tcjennings tcjennings deleted the tickets/DM-51396/v2_tables branch July 7, 2025 17:05
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