awatertrevi/infisical
1
0
Fork 0
mirror of https://github.com/awatertrevi/infisical.git synced 2026-10-08 14:28:20 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #1719 from Infisical/daniel/migration-fix

Browse Source 
Fix: Duplicate org membership migration
This commit is contained in:
Maidul Islam
2024-04-22 19:51:21 -04:00
committed by GitHub
parent 08cb105fe4 233a4f7d77
commit 1ba7a31e0d
1 changed files with 1 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
backend/src/db/migrations/20240405000045_org-memberships-unique-constraint.ts
+1
View File
@@ -42,6 +42,7 @@ export async function up(knex: Knex): Promise<void> {
await knex.transaction(async (tx) => { await knex.transaction(async (tx) => {
const duplicateRows = await tx(TableName.OrgMembership) const duplicateRows = await tx(TableName.OrgMembership)
.select("userId", "orgId") // Select the userId and orgId so we can group by them .select("userId", "orgId") // Select the userId and orgId so we can group by them
.whereNotNull("userId") // Ensure that the userId is not null
.count("* as cnt") // Count the number of rows for each userId and orgId, so we can make sure there are more than 1 row (a duplicate) .count("* as cnt") // Count the number of rows for each userId and orgId, so we can make sure there are more than 1 row (a duplicate)
.groupBy("userId", "orgId") .groupBy("userId", "orgId")
.havingRaw("count(*) > ?", [1]); // Using havingRaw for direct SQL expressions .havingRaw("count(*) > ?", [1]); // Using havingRaw for direct SQL expressions