@licklist/design 0.63.3-dev.3 → 0.63.3

package/src/recurring-date-picker-input/utils.ts

@@ -1,9 +1,4 @@
-import { TIME_FORMAT, TIME_FULL_FORMAT } from "@licklist/core/dist/Config";
-import { WorkHour } from "@licklist/core/dist/DataMapper/Provider/WorkHourDataMapper";
-import { fromFormat } from "@licklist/plugins/dist/utils/dateTime";
-import { DateTime, Interval } from "luxon";
-import { Validate } from "react-hook-form";
-import { useTranslation } from "react-i18next";
+import { DateTime } from "luxon";
 import RRule, { Frequency, Weekday } from "rrule";
 
 export const getWeekdayForFrequency = ({
@@ -16,7 +11,6 @@ export const getWeekdayForFrequency = ({
   const parsedDate = DateTime.fromISO(date);
   return new Weekday(
     parsedDate.weekday - 1,
-    // eslint-disable-next-line no-nested-ternary
     frequency !== Frequency.MONTHLY
       ? undefined
       : parsedDate.day + 7 > parsedDate.daysInMonth
@@ -115,86 +109,3 @@ export const parseAndValidateRRule = ({
     until: options.until,
   } as ParsedRRuleOptions;
 };
-
-export const useWorkHoursValidation = (
-  byWeekDay: Weekday[] = [],
-  workHours: WorkHour[] | undefined
-): {
-  validate?: Validate<string>;
-} => {
-  const { t } = useTranslation("Validation");
-  if (!workHours) return {};
-
-  const fromTimeFormat = fromFormat(TIME_FORMAT);
-  const fromTimeFullFormat = fromFormat(TIME_FULL_FORMAT);
-
-  // map every day to array of intervals describing workhours for that day
-  let intervalsByDay = workHours.map((workhour, index, self) => {
-    const res: Interval[] = [];
-
-    const startTime = fromTimeFormat(workhour.start);
-    const endTime = fromTimeFormat(workhour.end);
-
-    if (endTime > startTime) {
-      res.push(Interval.fromDateTimes(startTime, endTime));
-    } else {
-      // if end is before start, then it means that end is on the next day
-      // so count only to the end of current day
-      res.push(Interval.fromDateTimes(startTime, startTime.endOf("day")));
-    }
-
-    const prevIndex = index === 0 ? self.length - 1 : index - 1;
-
-    const prevStartTime = fromTimeFormat(self[prevIndex].start);
-    const prevEndTime = fromTimeFormat(self[prevIndex].end);
-
-    // if prevStartTime > prevEndTime then it means that prevEndTime is on the current day
-    // so we need to add interval from start of current day to prevEndTime
-    if (prevStartTime > prevEndTime) {
-      res.push(Interval.fromDateTimes(prevEndTime.startOf("day"), prevEndTime));
-    }
-
-    return res;
-  });
-
-  // if length === 0 or length === 7 then all workhours are selected, no need to filter
-  if (![0, 7].includes(byWeekDay.length)) {
-    intervalsByDay = intervalsByDay.filter((_, index) =>
-      byWeekDay.find(({ weekday }) => weekday === index)
-    );
-  }
-
-  return {
-    validate: (_time: string) => {
-      // workaround for issue with form values
-      // default values and entered values have different formats
-      const time = fromTimeFormat(_time).isValid
-        ? fromTimeFormat(_time)
-        : fromTimeFullFormat(_time);
-
-      const notFittingIntervalsByDay = intervalsByDay.reduce<Interval[][]>(
-        (acc, intervalsByDay) => {
-          if (!intervalsByDay.some((interval) => interval.contains(time))) {
-            acc.push(intervalsByDay);
-          }
-          return acc;
-        },
-        []
-      );
-
-      const notFittingIntervals = notFittingIntervalsByDay
-        .flat()
-        .filter((interval) => !interval.contains(time));
-
-      const attribute = notFittingIntervals
-        .map((interval) => interval.toFormat("HH:mm"))
-        .join(", ");
-
-      if (notFittingIntervalsByDay.length !== 0) {
-        return t("outsideWorkingHours", { attribute }) as string;
-      }
-
-      return true;
-    },
-  };
-};

package/src/sales/booking/results/components/ResultCard.tsx

@@ -12,6 +12,7 @@ import { TIME_FORMAT } from "@licklist/core/dist/Config";
 import * as Config from "@licklist/core/dist/Config";
 import { formatDateStringForEvent } from "@licklist/plugins/dist/utils/formatDate";
 import { getOrderSourceTitle } from "@licklist/plugins/dist/utils/sourceTitle";
+
 import Icon from "../../../../static/Icon";
 
 export type ResultCardProps = {
@@ -21,6 +22,8 @@ export type ResultCardProps = {
   onCardClick?: (id: number) => void;
 };
 
+// @TODO integrate when API will be available
+// activity and staff keys
 export const ResultCard = ({
   order,
   className,

package/src/setting/admin/AdminSettingForm.tsx

@@ -11,7 +11,7 @@ import Row from "react-bootstrap/Row";
 import Col from "react-bootstrap/Col";
 import { Link } from "react-router-dom";
 import HookFormService from "@licklist/plugins/dist/services/Form/HookFormService";
-import { ProvidableType } from "@licklist/core/dist/DataMapper/Provider/ProvidableDataMapper";
+import { ProviderType } from "@licklist/core/dist/DataMapper/Provider/ProvidableDataMapper";
 import Icon from "../../static/Icon";
 import { PaymentFeeForm, PaymentFeeFormFieldValues } from "./PaymentFeeForm";
 import { Currency } from "../../types/currency";
@@ -33,7 +33,7 @@ export interface AdminSettingFormProps extends HasPermissionProp {
   providerMetadata: {
     country?: string;
     providableId?: number;
-    providerType?: ProvidableType;
+    providerType?: ProviderType;
   };
 }
 

package/src/sortable-tree/SortableTreeItem.tsx

@@ -53,6 +53,7 @@ export interface SortableTreeItemProps {
   };
   setIsExpanded?: () => void;
   isOverride?: boolean;
+  isProduct?: boolean;
 }
 
 export function SortableTreeItem({
@@ -80,6 +81,7 @@ export function SortableTreeItem({
   secondaryBadge,
   setIsExpanded,
   isOverride,
+  isProduct = false,
 }: SortableTreeItemProps) {
   const [expanded, setExpanded] = useState(isExpanded);
   const [isModalVisible, setIsModalVisible] = useState(isNewAdded);
@@ -150,6 +152,7 @@ export function SortableTreeItem({
   };
 
   const onEdit = (e) => {
+    if (isOverride && !isProduct) return;
     e.stopPropagation();
     if (edit && !isModalVisible) edit();
     setIsModalVisible(true);

package/src/static/switch/BooleanSwitch.tsx

@@ -23,7 +23,7 @@ export function BooleanSwitch({
   const { t } = useTranslation("Design");
 
   const options: SelectItem[] = [
-    { id: 1, value: firstValueTitle ?? t("yes") },
+    { id: 1, value: firstValueTitle ??  t("yes") },
     { id: 0, value: secondValueTitle ?? t("no") },
   ];
 

package/src/styles/date-time-button/DateTimeButton.scss

@@ -81,7 +81,6 @@
     min-width: 5.3rem;
     max-width: 5.3rem;
     min-height: 2.75rem;
-    border: 2px solid $snippet-calendar-disabled-button-border-color;
   }
 
   &.calendar-view {

package/src/styles/iframe-page/PageBody.scss

@@ -65,6 +65,7 @@
       }
 
       @include media-breakpoint-down(sm) {
+        height: 100%;
         max-height: calc(100svh - 30rem);
       }
 

package/src/styles/ryft-payment-form/RyftPaymentForm.scss

@@ -135,14 +135,6 @@
       margin-bottom: 0.5rem;
     }
   }
-  .ryft-pay-grid {
-    display: flex;
-    flex-direction: column;
-    .apple-pay-button {
-      height: 2.5rem;
-      margin-top: 1rem;
-    }
-  }
   .mobile-pay-title {
     display: block;
     font-weight: 600;

package/src/styles/sales/BookingResults.scss

@@ -5,7 +5,7 @@
     flex-wrap: wrap;
 
     .result-card {
-      height: 14rem;
+      height: 14.25rem;
       border: 1px solid $gray-400;
       max-width: calc(50% - 0.75rem);
       flex: 0 0 calc(50% - 0.75rem);

package/src/styles/zones/ZoneForm.scss

@@ -1,5 +1,4 @@
 .zone-available-times {
-  overscroll-behavior: contain;
   display: flex;
   flex-wrap: wrap;
   gap: 1.5rem;
@@ -8,4 +7,4 @@
   justify-content: start;
   width: 95%;
   overflow: scroll;
-}
+}

package/src/zone/form/ZoneForm.tsx

@@ -13,7 +13,7 @@ export type ZoneFormProps = {
   onSubmit: (values: Partial<Zone>) => void;
   serverErrors?: ServerError;
   isLoading?: boolean;
-  workHours: WorkHour[] | undefined;
+  workHours?: WorkHour[];
 };
 
 export type ZoneFormValues = Partial<Zone> & {
@@ -25,7 +25,6 @@ export const ZoneForm = ({
   onSubmit,
   serverErrors,
   isLoading = false,
-  workHours,
 }: ZoneFormProps) => {
   const { t } = useTranslation("Design");
   const form = useForm<ZoneFormValues>({
@@ -55,7 +54,7 @@ export const ZoneForm = ({
   return (
     <FormProvider {...form}>
       <Form onSubmit={handleSubmit(onFormSubmit)}>
-        <ZoneControl isLoading={isLoading} workHours={workHours} />
+        <ZoneControl isLoading={isLoading} />
         <Row>
           <Col md={6} sm={12} />
           <Col

package/src/zone/form/components/ZoneControl.tsx

@@ -11,15 +11,15 @@ import { GameDurationControl } from "./GameDurationControl";
 
 export type ZoneControlProps = {
   isLoading: boolean;
-  workHours: WorkHour[] | undefined;
+  workHours?: WorkHour[];
 };
 
-export const ZoneControl = ({ isLoading, workHours }: ZoneControlProps) => {
+export const ZoneControl = ({ isLoading }: ZoneControlProps) => {
   return (
     <Row>
       <Col md={6} sm={12} className="pr-5">
         <NameControl isLoading={isLoading} />
-        <ZoneRecurrencesControl workHours={workHours} />
+        <ZoneRecurrencesControl />
         <hr />
         <Row>
           <Col md={6} sm={6}>

package/src/zone/form/components/ZoneRecurrencesControl.tsx

@@ -25,13 +25,12 @@ export interface DateAndRecurrenceInputValues {
   zoneRecurrences?: Partial<ZoneRecurrence>[];
 }
 
-type ZoneRecurrenceControlProps = {
-  workHours: WorkHour[] | undefined;
+export type ZoneRecurrenceControlProps = {
+  workHours?: WorkHour[];
+  isLoading?: boolean;
 };
 
-export const ZoneRecurrencesControl = ({
-  workHours,
-}: ZoneRecurrenceControlProps) => {
+export const ZoneRecurrencesControl = () => {
   const {
     control,
     register,
@@ -70,7 +69,7 @@ export const ZoneRecurrencesControl = ({
     clearEditState();
   });
 
-  const popoverId = useId();
+  const popoverId = useId()!;
 
   const { t } = useTranslation(["Design", "Validation"]);
 
@@ -167,7 +166,6 @@ export const ZoneRecurrencesControl = ({
                 onChange={handleRecurringDateChange}
                 onDelete={handleDelete}
                 setInitialStartDateAfterSelect
-                workHours={workHours}
               />
             </Popover.Content>
           </Popover>

package/src/zone/form/utils/dates.ts

@@ -53,17 +53,18 @@ export const getCheckboxesByWorkHours = (workHours?: WorkHour[]) => {
   if (!workHours || !workHours.length)
     return getTimeCheckboxRanges(DEFAULT_START_DAY_TIME, DEFAULT_END_DAY_TIME);
 
-  const providerWorkHoursByWeek = workHours.reduce<
-    { start: string; end: string }[]
-  >((workHoursNextArray, currentWorkHour) => {
-    const { start, end } = currentWorkHour;
+  const providerWorkHoursByWeek = workHours.reduce(
+    (workHoursNextArray: { start: string; end: string }[], currentWorkHour) => {
+      const { start, end } = currentWorkHour;
 
-    if (start && end) {
-      return [...workHoursNextArray, { start, end }];
-    }
+      if (start && end) {
+        return [...workHoursNextArray, { start, end }];
+      }
 
-    return workHoursNextArray;
-  }, []);
+      return workHoursNextArray;
+    },
+    []
+  );
 
   // If venue has filled only 1 work day from wee
   // For another days we use default 00:00 and 23:59

package/tests/Auth/Authorizer.test.tsx

@@ -0,0 +1,194 @@
+import RouteService from "@licklist/plugins/dist/services/Route/RouteService";
+import { mount, shallow } from "enzyme";
+import * as React from "react";
+import { act } from "react-dom/test-utils";
+import { QueryClientProvider } from "react-query";
+import { Authorizer } from "../../src/auth/Authorizer";
+import useAuthMock from "../__mock__/hooks/useAuthMock";
+import useUserMock from "../__mock__/hooks/useUserMock";
+import useAuthApi from "../__mock__/hooks/useAuthApi";
+import { queryClient } from "../packages/react-query";
+
+jest.mock("react-i18next", () => ({
+  Trans: jest.fn().mockReturnValue("test"),
+  useTranslation: () => ({
+    t: jest.fn((arg) => arg),
+  }),
+}));
+jest.mock("@licklist/plugins/dist/context/user/hooks/useAuth");
+jest.mock("@licklist/plugins/dist/context/user/hooks/useUser");
+jest.mock("@licklist/plugins/dist/hooks/Api/useAuthApi");
+
+const DEFAULT_PROFILE_RESPONSE = {
+  useProfile: () => ({
+    isLoading: false,
+    isFetching: false,
+    isError: false,
+    data: { access_token: "test" },
+  }),
+};
+
+const DEFAULT_PROFILE_LOADING_RESPONSE = {
+  useProfile: () => ({
+    isLoading: true,
+    isFetching: false,
+    isError: false,
+    data: { access_token: "test" },
+  }),
+};
+
+describe("Authorizer", () => {
+  it("should be rendered without issues", () => {
+    useAuthMock({
+      secrets: null,
+    });
+    useUserMock({
+      profile: null,
+    });
+    useAuthApi(DEFAULT_PROFILE_LOADING_RESPONSE);
+
+    const component = shallow(
+      <QueryClientProvider client={queryClient}>
+        <Authorizer />
+      </QueryClientProvider>
+    );
+
+    expect(component.exists()).toBeTruthy();
+  });
+
+  it("should render children if authorized", () => {
+    let component: any;
+
+    useAuthMock({
+      secrets: "not-null",
+    });
+    useUserMock({
+      profile: "not-null",
+    });
+    useAuthApi(DEFAULT_PROFILE_LOADING_RESPONSE);
+
+    RouteService.getCurrentPage = jest.fn();
+    RouteService.getCurrentRoute = jest.fn();
+    RouteService.cleanRedirect = jest.fn();
+
+    act(() => {
+      component = mount(
+        <QueryClientProvider client={queryClient}>
+          <Authorizer>
+            <p>test</p>
+          </Authorizer>
+        </QueryClientProvider>
+      );
+    });
+
+    component.update();
+
+    expect(RouteService.getCurrentPage).toHaveBeenCalledTimes(0);
+    expect(RouteService.getCurrentRoute).toHaveBeenCalledTimes(0);
+    expect(RouteService.cleanRedirect).toHaveBeenCalledTimes(0);
+    expect(component.exists()).toBeTruthy();
+    expect(component.find("BlockLoader").exists()).toBeTruthy();
+  });
+
+  it("should render children if current page is login", () => {
+    let component: any;
+
+    useAuthMock({
+      secrets: null,
+    });
+    useUserMock({
+      profile: null,
+    });
+    useAuthApi(DEFAULT_PROFILE_LOADING_RESPONSE);
+
+    RouteService.getCurrentPage = jest.fn().mockReturnValue("/login");
+    RouteService.getCurrentRoute = jest.fn();
+    RouteService.cleanRedirect = jest.fn();
+
+    act(() => {
+      component = mount(
+        <QueryClientProvider client={queryClient}>
+          <Authorizer>
+            <p>test</p>
+          </Authorizer>
+        </QueryClientProvider>
+      );
+    });
+
+    component.update();
+
+    expect(RouteService.getCurrentPage).toHaveBeenCalledTimes(0);
+    expect(RouteService.getCurrentRoute).toHaveBeenCalledTimes(0);
+    expect(RouteService.cleanRedirect).toHaveBeenCalledTimes(0);
+    expect(component.exists()).toBeTruthy();
+    expect(component.find("BlockLoader").exists()).toBeTruthy();
+  });
+
+  it("should NOT render children if profile is loading or fetching", () => {
+    let component: any;
+
+    useAuthMock({
+      secrets: null,
+    });
+    useUserMock({
+      profile: null,
+    });
+    useAuthApi(DEFAULT_PROFILE_LOADING_RESPONSE);
+
+    RouteService.getCurrentPage = jest.fn();
+    RouteService.getCurrentRoute = jest.fn();
+    RouteService.cleanRedirect = jest.fn();
+
+    act(() => {
+      component = mount(
+        <QueryClientProvider client={queryClient}>
+          <Authorizer>
+            <p>test</p>
+          </Authorizer>
+        </QueryClientProvider>
+      );
+    });
+
+    component.update();
+
+    expect(RouteService.getCurrentPage).toHaveBeenCalledTimes(0);
+    expect(RouteService.getCurrentRoute).toHaveBeenCalledTimes(0);
+    expect(RouteService.cleanRedirect).toHaveBeenCalledTimes(0);
+    expect(component.exists()).toBeTruthy();
+    expect(component.find("BlockLoader").exists()).toBeTruthy();
+  });
+
+  it("should NOT render children if unauthorized and current page is not login", () => {
+    let component: any;
+
+    useAuthMock({
+      secrets: null,
+    });
+    useUserMock({
+      profile: null,
+    });
+    useAuthApi(DEFAULT_PROFILE_RESPONSE);
+
+    RouteService.getCurrentPage = jest.fn();
+    RouteService.getCurrentRoute = jest.fn();
+    RouteService.cleanRedirect = jest.fn();
+
+    act(() => {
+      component = mount(
+        <QueryClientProvider client={queryClient}>
+          <Authorizer>
+            <p>test</p>
+          </Authorizer>
+        </QueryClientProvider>
+      );
+    });
+
+    component.update();
+
+    expect(RouteService.getCurrentPage).toHaveBeenCalledTimes(3);
+    expect(RouteService.getCurrentRoute).toHaveBeenCalledTimes(1);
+    expect(RouteService.cleanRedirect).toHaveBeenCalledTimes(1);
+    expect(component.exists()).toBeTruthy();
+    expect(component.find("BlockLoader").exists()).toBeTruthy();
+  });
+});

package/tests/Auth/Layout/UserNavDropDown.test.tsx

@@ -0,0 +1,43 @@
+import RouteService from "@licklist/plugins/dist/services/Route/RouteService";
+import { mount, shallow } from "enzyme";
+import * as React from "react";
+import useTranslationMock from "../../__mock__/hooks/useTranslationMock";
+import useUserMock from "../../__mock__/hooks/useUserMock";
+import { UserNavDropDown } from "../../../src/auth/Layout/UserNavDropDown";
+
+jest.mock("react-i18next");
+jest.mock("@licklist/plugins/dist/context/user/hooks/useUser");
+
+describe("UserNavDropDown", () => {
+  it("should be rendered without issues", () => {
+    useTranslationMock({
+      t: jest.fn(),
+    });
+    useUserMock({
+      profile: null,
+    });
+
+    const component = shallow(<UserNavDropDown />);
+
+    expect(component.exists()).toBeTruthy();
+  });
+
+  it("should logout on click", () => {
+    useTranslationMock({
+      t: jest.fn(),
+    });
+    useUserMock({
+      logout: jest.fn(),
+    });
+
+    RouteService.redirectTo = jest.fn();
+
+    const component = mount(<UserNavDropDown />);
+
+    expect(component.exists()).toBeTruthy();
+
+    component.children().prop("onClick")();
+
+    expect(RouteService.redirectTo).toHaveBeenCalledTimes(1);
+  });
+});

package/tests/Auth/Layout/UserNavDropDownToggle.test.tsx

@@ -0,0 +1,33 @@
+import { IUser } from "@licklist/plugins/dist/types/context/user/IUser";
+import { shallow } from "enzyme";
+import * as React from "react";
+import { UserNavDropDownToggle } from "../../../src/auth/Layout/UserNavDropDownToggle";
+import useTranslationMock from "../../__mock__/hooks/useTranslationMock";
+
+jest.mock("react-i18next");
+jest.mock("@licklist/plugins/dist/context/user/hooks/useUser");
+
+const user: IUser = {
+  id: 1,
+  first_name: "first_name",
+  last_name: "last_name",
+  email: "email",
+  user_roles: [],
+  user_detail: {
+    postcode: "",
+    birthday: "",
+    phone: "",
+  },
+};
+
+describe("UserNavDropDownToggle", () => {
+  it("should be rendered without issues", () => {
+    useTranslationMock({
+      t: jest.fn(),
+    });
+
+    const component = shallow(<UserNavDropDownToggle user={user} />);
+
+    expect(component.exists()).toBeTruthy();
+  });
+});