-
-
Notifications
You must be signed in to change notification settings - Fork 1
Closed
Labels
Description
Problem
PRs #2532 and #2551 replaced postgresql.UUID with sqlalchemy.types.Uuid in 11 model files. However, 9 more files in autobot-backend/models/ still import UUID from sqlalchemy.dialects.postgresql alongside JSONB:
autobot-backend/models/process_run.pyautobot-backend/models/workflow_audit.pyautobot-backend/models/session_collaboration.pyautobot-backend/models/activities.pyautobot-backend/models/heartbeat.pyautobot-backend/models/task_delegation.pyautobot-backend/models/config_revision.pyautobot-backend/models/approval.pyautobot-backend/models/secret.py
These files need the same import-splitting treatment as #2533: keep JSONB from postgresql, move UUID to sqlalchemy.types.Uuid.
Discovered During
Code review of PR #2551 (#2533)
Fix
Apply the same pattern: split from sqlalchemy.dialects.postgresql import JSONB, UUID into two imports, replacing UUID with from sqlalchemy.types import Uuid.
Impact
Breaks SQLite dev/test environments. Medium priority — same as #2495 and #2533.
Reactions are currently unavailable