Skip to content

Commit

Permalink
Address PR feedback
Browse files Browse the repository at this point in the history
  • Loading branch information
motechFR committed Aug 21, 2024
1 parent c7c3dbc commit 23e257e
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 21 deletions.
2 changes: 1 addition & 1 deletion @connect-shared/lib/projects/createProject.ts
Original file line number Diff line number Diff line change
Expand Up @@ -78,7 +78,7 @@ export async function createProject({
updatedBy: userId,
userId: member.userId,
// This is necessary because some test data fids do not have a corresponding farcaster profile
name: member.account.displayName || member.account.username,
name: (member.account.displayName || member.account.username)?.trim() || '',
farcasterId: member.fid
}))
];
Expand Down
22 changes: 2 additions & 20 deletions @connect-shared/lib/projects/editProject.ts
Original file line number Diff line number Diff line change
@@ -1,15 +1,9 @@
import { InvalidInputError } from '@charmverse/core/errors';
import { log } from '@charmverse/core/log';
import type { Project } from '@charmverse/core/prisma-client';
import { prisma } from '@charmverse/core/prisma-client';
import type { StatusAPIResponse } from '@farcaster/auth-kit';
import { resolveENSName } from '@root/lib/blockchain';
import { ensureFarcasterUserExists } from '@root/lib/farcaster/ensureFarcasterUserExists';
import { getFarcasterUsers } from '@root/lib/farcaster/getFarcasterUsers';
import { uid } from '@root/lib/utils/strings';
import { isTruthy } from '@root/lib/utils/types';
import { meanBy } from 'lodash';
import { v4 } from 'uuid';

import type { FormValues } from './projectSchema';

Expand All @@ -29,7 +23,7 @@ export async function editProject({ userId, input }: { input: EditProjectValues;
}
}

const [currentProjectMembers, inputProjectMembers] = await Promise.all([
const [currentProjectMembers] = await Promise.all([
prisma.projectMember.findMany({
where: {
projectId: input.projectId,
Expand All @@ -47,18 +41,6 @@ export async function editProject({ userId, input }: { input: EditProjectValues;
}
}
}
}),
prisma.farcasterUser.findMany({
where: {
fid: {
in: input.projectMembers.map(({ farcasterId }) => farcasterId)
}
},
select: {
userId: true,
fid: true,
account: true
}
})
]);

Expand All @@ -73,7 +55,7 @@ export async function editProject({ userId, input }: { input: EditProjectValues;
);

const projectMembers = await Promise.all(
newProjectMembers.map(async (member) => ensureFarcasterUserExists({ fid: member.farcasterId }))
newProjectMembers.map((member) => ensureFarcasterUserExists({ fid: member.farcasterId }))
);

const [editedProject] = await prisma.$transaction([
Expand Down

0 comments on commit 23e257e

Please sign in to comment.