Fix user email unique constraint

[Weiko]

Context

Fix wrong unique index on user email as we don't want an index on email/deletedAt but a partial on a where condition on deletedAt. This should enforce email unicity excluding the ones that have a deletedAt

Test

Run

SELECT email, COUNT(*) as duplicate_count
FROM core."user"
WHERE "deletedAt" IS NULL
GROUP BY email
HAVING COUNT(*) > 1
ORDER BY duplicate_count DESC;

to check duplicates before running the migration

[greptile-apps]

PR Summary

Modified the user email uniqueness constraint to allow reuse of email addresses for soft-deleted users while maintaining uniqueness for active users.

• Changed unique constraint in /packages/twenty-server/src/database/typeorm/core/migrations/1733408604468-fixUserEmailUniqueConstraint.ts from composite (email, deletedAt) to partial index on email where deletedAt IS NULL
• Updated /packages/twenty-server/src/engine/core-modules/user/user.entity.ts to use @Index decorator with WHERE clause instead of @unique for email constraint
• Migration includes rollback functionality to restore original composite unique constraint if needed

_{2 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile}