Skip to content

fix(models): replace postgresql.UUID in 6 more user_management models (#2533)#2551

Merged
mrveiss merged 1 commit intoDev_new_guifrom
fix/issue-2533
Mar 27, 2026
Merged

fix(models): replace postgresql.UUID in 6 more user_management models (#2533)#2551
mrveiss merged 1 commit intoDev_new_guifrom
fix/issue-2533

Conversation

@mrveiss
Copy link
Copy Markdown
Owner

@mrveiss mrveiss commented Mar 27, 2026

Summary

  • Splits JSONB/UUID imports in 6 user_management model files that previously imported both from sqlalchemy.dialects.postgresql
  • Replaces postgresql.UUID with dialect-agnostic sqlalchemy.types.Uuid while keeping postgresql.JSONB imports
  • Files: user.py, organization.py, team.py, sso.py, audit.py, api_key.py
  • Follows the same pattern as PR fix(models): replace postgresql.UUID with database-agnostic UUID (#2495) #2532 which fixed the first 5 model files

Closes #2533

@github-actions
Copy link
Copy Markdown

⚠️ SSOT Configuration Compliance: Violations Found

Metric Count
Total Violations 2
SSOT Violations (high priority) 1
Other Violations 1

⚠️ 1 values have SSOT config equivalents!

These should be replaced with SSOT config imports:

Python:

from src.config.ssot_config import config
# Use: config.vm.main, config.port.backend, config.backend_url

TypeScript:

import config from '@/config/ssot-config'
// Use: config.vm.main, config.port.backend, config.backendUrl

📖 See SSOT_CONFIG_GUIDE.md for documentation.

@mrveiss mrveiss merged commit fa5c405 into Dev_new_gui Mar 27, 2026
7 of 9 checks passed
@mrveiss mrveiss deleted the fix/issue-2533 branch March 27, 2026 13:29
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.

1 participant