Fixed ministry memeber check

Author: jzongker

src/helpers/SecondaryMenuHelper.ts

@@ -10,7 +10,7 @@ export class SecondaryMenuHelper {
   static getSecondaryMenu = (path: string, data: any) => {
     let result: { menuItems: MenuItem[]; label: string } = { menuItems: [], label: "" };
 
-    if (path.startsWith("/people") || path.startsWith("/groups") || path.startsWith("/attendance")) result = this.getPeopleMenu(path, data.search);
+    if (path.startsWith("/people") || path.startsWith("/groups") || path.startsWith("/attendance")) result = this.getPeopleMenu(path, data.search, data?.isMinistryMember);
     else if (path.startsWith("/mobile")) result = this.getMobileMenu(path);
     else if (path.startsWith("/settings") || path.startsWith("/admin") || path.startsWith("/forms")) result = this.getSettingsMenu(path, data);
     else if (path.startsWith("/serving")) result = this.getServingMenu(path, data);
@@ -22,11 +22,11 @@ export class SecondaryMenuHelper {
     return result;
   };
 
-  static getPeopleMenu = (path: string, search?: string) => {
+  static getPeopleMenu = (path: string, search?: string, isMinistryMember?: boolean) => {
     const menuItems: MenuItem[] = [];
     let label: string = "";
     menuItems.push({ url: "/groups", label: Locale.label("components.wrapper.groups"), icon: "groups" });
-    if (UserHelper.checkAccess(Permissions.membershipApi.plans.edit) || data?.isMinistryMember) menuItems.push({ url: "/serving", label: Locale.label("components.wrapper.teams"), icon: "people" });
+    if (UserHelper.checkAccess(Permissions.membershipApi.plans.edit) || isMinistryMember) menuItems.push({ url: "/serving", label: Locale.label("components.wrapper.teams"), icon: "people" });
     menuItems.push({ url: "/people", label: Locale.label("components.wrapper.ppl"), icon: "person" });
     if (UserHelper.checkAccess(Permissions.attendanceApi.attendance.viewSummary)) menuItems.push({ url: "/attendance", label: Locale.label("components.wrapper.att"), icon: "calendar_month" });