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.

Sign up for GitHub

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

Jump to bottom

Replace TaskInstanceNote composite primary key with TI.id #47376

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

Replace TaskInstanceNote composite primary key with TI.id #47376

wants to merge 10 commits into from
+1,544 −1,432

Conversation

ephraimbuddy
Copy link
Contributor

This commit removes the composite primary key on TaskInstanceNote and replaces it with TaskInstance ID key, which is a UUID.

Related: #44147

@ephraimbuddy ephraimbuddy force-pushed the fk-taskinstance-note-ti branch from 99e7db3 to f6fd8f0 Compare March 5, 2025 15:09
@ephraimbuddy ephraimbuddy force-pushed the fk-taskinstance-note-ti branch from f6fd8f0 to 5853c0e Compare March 6, 2025 17:16
ephraimbuddy
ephraimbuddy commented Mar 6, 2025
View reviewed changes
@ephraimbuddy ephraimbuddy force-pushed the fk-taskinstance-note-ti branch from c01388e to 5c82082 Compare March 6, 2025 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jedcunningham jedcunningham jedcunningham left review comments

@potiuk potiuk Awaiting requested review from potiuk potiuk is a code owner

@XD-DENG XD-DENG Awaiting requested review from XD-DENG XD-DENG is a code owner

@ashb ashb Awaiting requested review from ashb ashb is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
area:db-migrations PRs with DB migration kind:documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ephraimbuddy @jedcunningham