Merge pull request #3370 from akhilmhdh/fix/doc-p-access-image

added check for org in project id
This commit is contained in:
Maidul Islam
2025-04-07 18:24:08 +01:00
committed by GitHub
@@ -18,7 +18,7 @@ import { TAccessProjectDTO, TListOrgProjectsDTO } from "./org-admin-types";
type TOrgAdminServiceFactoryDep = { type TOrgAdminServiceFactoryDep = {
permissionService: Pick<TPermissionServiceFactory, "getOrgPermission">; permissionService: Pick<TPermissionServiceFactory, "getOrgPermission">;
projectDAL: Pick<TProjectDALFactory, "find" | "findById" | "findProjectGhostUser">; projectDAL: Pick<TProjectDALFactory, "find" | "findById" | "findProjectGhostUser" | "findOne">;
projectMembershipDAL: Pick< projectMembershipDAL: Pick<
TProjectMembershipDALFactory, TProjectMembershipDALFactory,
"findOne" | "create" | "transaction" | "delete" | "findAllProjectMembers" "findOne" | "create" | "transaction" | "delete" | "findAllProjectMembers"
@@ -95,7 +95,7 @@ export const orgAdminServiceFactory = ({
OrgPermissionSubjects.AdminConsole OrgPermissionSubjects.AdminConsole
); );
const project = await projectDAL.findById(projectId); const project = await projectDAL.findOne({ id: projectId, orgId: actorOrgId });
if (!project) throw new NotFoundError({ message: `Project with ID '${projectId}' not found` }); if (!project) throw new NotFoundError({ message: `Project with ID '${projectId}' not found` });
if (project.version === ProjectVersion.V1) { if (project.version === ProjectVersion.V1) {