Fix unique index email people

[Weiko]

Context

WorkspaceIsUnique decorator allows us to create unique indexes on our tables. Here Emails is a composite field containing a TEXT subfield. Due to the fact that TEXT fields are non-nullable and have empty strings as default values, adding a unique index on a TEXT field or any composite containing TEXT as subfields will fail and throw if we try to create more than 1 record that does not specify a value to the TEXT field.

This PR simply removes the index for the time being until we find a solution

[greptile-apps]

PR Summary

This pull request addresses an issue with the unique index on the 'emails' field in the PersonWorkspaceEntity by temporarily removing the @WorkspaceIsUnique decorator.

• Commented out @WorkspaceIsUnique decorator for 'emails' field in packages/twenty-server/src/modules/person/standard-objects/person.workspace-entity.ts
• Removal of unique index may allow duplicate email entries in the database
• Temporary fix until a solution for handling TEXT fields with empty string defaults is found
• Potential impact on data integrity and uniqueness constraints for person records
• Consider implementing application-level email validation to mitigate duplicate entries

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