Skip to content

Chore/update unique pattern - #266

Merged
t-a-y-l-o-r merged 3 commits into
developfrom
chore/update-unique-pattern
Sep 16, 2026
Merged

Chore/update unique pattern#266
t-a-y-l-o-r merged 3 commits into
developfrom
chore/update-unique-pattern

Conversation

@t-a-y-l-o-r

@t-a-y-l-o-r t-a-y-l-o-r commented Sep 16, 2026 •
edited by cubic-dev-ai Bot
Loading

Copy link
Copy Markdown
Collaborator

closes #145

Summary by cubic

Replaces AssetCustomField’s legacy unique_together setting with a named UniqueConstraint and changes value_text from nullable to non-null with an empty-string default. This keeps each asset/field pair unique while making blank values consistent in current and historical records.

Migration

  • Apply migration 0013 to update the database schema.

Written for commit 451b401. Summary will update on new commits.

Review in cubic

@t-a-y-l-o-r t-a-y-l-o-r self-assigned this Sep 16, 2026
cubic-dev-ai[bot]
cubic-dev-ai Bot reviewed Sep 16, 2026
View reviewed changes

@cubic-dev-ai cubic-dev-ai Bot left a comment •
edited
Loading

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All reported issues were addressed across 2 files

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread blueflow/models/asset_custom_field.py
@t-a-y-l-o-r
t-a-y-l-o-r merged commit 5fce85b into develop Sep 16, 2026
3 of 4 checks passed
@t-a-y-l-o-r
t-a-y-l-o-r deleted the chore/update-unique-pattern branch September 16, 2026 17:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cubic-dev-ai cubic-dev-ai[bot] cubic-dev-ai[bot] left review comments

Assignees

@t-a-y-l-o-r t-a-y-l-o-r

Labels

cleanup

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

chore: modernize legacy unique_together to UniqueConstraint

1 participant

@t-a-y-l-o-r