🐛 Don't require organization ID for memberships

staart · May 12, 2019 · 7cdf593 · 7cdf593
1 parent 00d5a2a
commit 7cdf593
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 9 deletions.
diff --git a/src/rest/membership.ts b/src/rest/membership.ts
@@ -22,8 +22,7 @@ import { createEvent } from "../crud/event";
 
 export const getMembershipDetailsForUser = async (
   userId: number,
-  membershipId: number,
-  organizationId: number
+  membershipId: number
 ) => {
   if (await can(userId, Authorizations.READ, "membership", membershipId))
     return await getMembership(membershipId);

diff --git a/src/routes/membership.ts b/src/routes/membership.ts
@@ -9,14 +9,9 @@ import { getOrganizationMemberDetails } from "../crud/membership";
 
 export const routeMembershipGet = async (req: Request, res: Response) => {
   const id = req.params.id;
-  const organizationId = req.params.organizationId;
-  if (!id || !organizationId) throw new Error(ErrorCode.MISSING_FIELD);
+  if (!id) throw new Error(ErrorCode.MISSING_FIELD);
   res.json({
-    membership: await getMembershipDetailsForUser(
-      res.locals.token.id,
-      id,
-      organizationId
-    )
+    membership: await getMembershipDetailsForUser(res.locals.token.id, id)
   });
 };