From a41dd30467d3659aebb066c64d415efb3bd57fa9 Mon Sep 17 00:00:00 2001
From: Syed Ali Shahbaz <52925846+alishaz-polymath@users.noreply.github.com>
Date: Wed, 9 Feb 2022 23:55:58 +0530
Subject: [PATCH] Add reason for rejection (optional) (#1719)

* init --WIP

* added rejection option modal

* migration added

* lint fix

* rejection reason in email added

* moved getRejectedReason function

* lint fix

* --wip

* Prevent undefineds and nulls on email messages

* Cleanup

Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
Co-authored-by: zomars <zomars@me.com>
---
 @types/@wojtekmaj:react-daterange-picker.d.ts |  15 --
 components/booking/BookingListItem.tsx        | 162 +++++++++++-------
 .../templates/attendee-declined-email.ts      |   2 +
 .../templates/attendee-scheduled-email.ts     |  11 ++
 .../calendar/interfaces/Calendar.ts           |   1 +
 pages/api/book/confirm.ts                     |   4 +-
 .../migration.sql                             |   2 +
 prisma/schema.prisma                          |   1 +
 prisma/zod/booking.ts                         |   1 +
 public/static/locales/en/common.json          |   4 +
 10 files changed, 128 insertions(+), 75 deletions(-)
 delete mode 100644 @types/@wojtekmaj:react-daterange-picker.d.ts
 create mode 100644 prisma/migrations/20220209082843_add_rejection_reason/migration.sql

diff --git a/@types/@wojtekmaj:react-daterange-picker.d.ts b/@types/@wojtekmaj:react-daterange-picker.d.ts
deleted file mode 100644
index 83839408..00000000
--- a/@types/@wojtekmaj:react-daterange-picker.d.ts
+++ /dev/null
@@ -1,15 +0,0 @@
-declare module "@wojtekmaj/react-daterange-picker/dist/entry.nostyle" {
-  import { CalendarProps } from "react-calendar";
-  export type DateRangePickerCalendarProps = Omit<
-    CalendarProps,
-    "calendarClassName" | "onChange" | "value"
-  > & {
-    calendarClassName?: string;
-    onChange: (value: [Date, Date]) => void;
-    value: [Date, Date];
-    clearIcon: JSX.Element | null;
-    calendarIcon: JSX.Element | null;
-    rangeDivider: JSX.Element | null;
-  };
-  export default function DateRangePicker(props: DateRangePickerCalendarProps): JSX.Element;
-}
diff --git a/components/booking/BookingListItem.tsx b/components/booking/BookingListItem.tsx
index 45efad84..edcf73f7 100644
--- a/components/booking/BookingListItem.tsx
+++ b/components/booking/BookingListItem.tsx
@@ -1,25 +1,35 @@
 import { BanIcon, CheckIcon, ClockIcon, XIcon } from "@heroicons/react/outline";
 import { BookingStatus } from "@prisma/client";
 import dayjs from "dayjs";
+import { useState } from "react";
 import { useMutation } from "react-query";
 
 import { HttpError } from "@lib/core/http/error";
 import { useLocale } from "@lib/hooks/useLocale";
 import { inferQueryOutput, trpc } from "@lib/trpc";
 
+import { TextArea } from "@components/form/fields";
 import TableActions, { ActionType } from "@components/ui/TableActions";
+import { Dialog, DialogClose, DialogContent, DialogFooter, DialogHeader } from "@components/Dialog";
+import Button from "@components/ui/Button";
 
 type BookingItem = inferQueryOutput<"viewer.bookings">["bookings"][number];
 
 function BookingListItem(booking: BookingItem) {
   const { t, i18n } = useLocale();
   const utils = trpc.useContext();
-
+  const [rejectionReason, setRejectionReason] = useState<string>("");
+  const [rejectionDialogIsOpen, setRejectionDialogIsOpen] = useState(false);
   const mutation = useMutation(
     async (confirm: boolean) => {
       const res = await fetch("/api/book/confirm", {
         method: "PATCH",
-        body: JSON.stringify({ id: booking.id, confirmed: confirm, language: i18n.language }),
+        body: JSON.stringify({
+          id: booking.id,
+          confirmed: confirm,
+          language: i18n.language,
+          reason: rejectionReason,
+        }),
         headers: {
           "Content-Type": "application/json",
         },
@@ -41,7 +51,7 @@ function BookingListItem(booking: BookingItem) {
     {
       id: "reject",
       label: t("reject"),
-      onClick: () => mutation.mutate(false),
+      onClick: () => setRejectionDialogIsOpen(true),
       icon: BanIcon,
       disabled: mutation.isLoading,
     },
@@ -73,64 +83,98 @@ function BookingListItem(booking: BookingItem) {
   const startTime = dayjs(booking.startTime).format(isUpcoming ? "ddd, D MMM" : "D MMMM YYYY");
 
   return (
-    <tr className="flex">
-      <td className="hidden whitespace-nowrap py-4 align-top ltr:pl-6 rtl:pr-6 sm:table-cell">
-        <div className="text-sm leading-6 text-gray-900">{startTime}</div>
-        <div className="text-sm text-gray-500">
-          {dayjs(booking.startTime).format("HH:mm")} - {dayjs(booking.endTime).format("HH:mm")}
-        </div>
-      </td>
-      <td className={"flex-1 py-4 ltr:pl-4 rtl:pr-4" + (booking.rejected ? " line-through" : "")}>
-        <div className="sm:hidden">
-          {!booking.confirmed && !booking.rejected && (
-            <Tag className="mb-2 ltr:mr-2 rtl:ml-2">{t("unconfirmed")}</Tag>
-          )}
-          {!!booking?.eventType?.price && !booking.paid && (
-            <Tag className="mb-2 ltr:mr-2 rtl:ml-2">Pending payment</Tag>
-          )}
-          <div className="text-sm font-medium text-gray-900">
-            {startTime}:{" "}
-            <small className="text-sm text-gray-500">
-              {dayjs(booking.startTime).format("HH:mm")} - {dayjs(booking.endTime).format("HH:mm")}
-            </small>
-          </div>
-        </div>
-        <div
-          title={booking.title}
-          className="max-w-56 truncate text-sm font-medium leading-6 text-neutral-900 md:max-w-max">
-          {booking.eventType?.team && <strong>{booking.eventType.team.name}: </strong>}
-          {booking.title}
-          {!!booking?.eventType?.price && !booking.paid && (
-            <Tag className="hidden ltr:ml-2 rtl:mr-2 sm:inline-flex">Pending payment</Tag>
-          )}
-          {!booking.confirmed && !booking.rejected && (
-            <Tag className="hidden ltr:ml-2 rtl:mr-2 sm:inline-flex">{t("unconfirmed")}</Tag>
-          )}
-        </div>
-        {booking.description && (
-          <div className="max-w-52 truncate text-sm text-gray-500 md:max-w-96" title={booking.description}>
-            &quot;{booking.description}&quot;
-          </div>
-        )}
-        {booking.attendees.length !== 0 && (
-          <div className="text-sm text-gray-900 hover:text-blue-500">
-            <a href={"mailto:" + booking.attendees[0].email}>{booking.attendees[0].email}</a>
-          </div>
-        )}
-      </td>
+    <>
+      <Dialog open={rejectionDialogIsOpen} onOpenChange={setRejectionDialogIsOpen}>
+        <DialogContent>
+          <DialogHeader title={t("rejection_reason_title")} />
 
-      <td className="whitespace-nowrap py-4 text-right text-sm font-medium ltr:pr-4 rtl:pl-4">
-        {isUpcoming && !isCancelled ? (
-          <>
-            {!booking.confirmed && !booking.rejected && <TableActions actions={pendingActions} />}
-            {booking.confirmed && !booking.rejected && <TableActions actions={bookedActions} />}
-            {!booking.confirmed && booking.rejected && (
-              <div className="text-sm text-gray-500">{t("rejected")}</div>
+          <p className="-mt-4 text-sm text-gray-500">{t("rejection_reason_description")}</p>
+          <p className="mt-6 mb-2 text-sm font-bold text-black">
+            {t("rejection_reason")}
+            <span className="font-normal text-gray-500"> (Optional)</span>
+          </p>
+          <TextArea
+            name={t("rejection_reason")}
+            value={rejectionReason}
+            onChange={(e) => setRejectionReason(e.target.value)}
+            className="mb-5 sm:mb-6"
+          />
+
+          <DialogFooter>
+            <DialogClose>
+              <Button color="secondary">{t("cancel")}</Button>
+            </DialogClose>
+
+            <Button
+              disabled={mutation.isLoading}
+              onClick={() => {
+                mutation.mutate(false);
+              }}>
+              {t("rejection_confirmation")}
+            </Button>
+          </DialogFooter>
+        </DialogContent>
+      </Dialog>
+
+      <tr className="flex">
+        <td className="hidden whitespace-nowrap py-4 align-top ltr:pl-6 rtl:pr-6 sm:table-cell">
+          <div className="text-sm leading-6 text-gray-900">{startTime}</div>
+          <div className="text-sm text-gray-500">
+            {dayjs(booking.startTime).format("HH:mm")} - {dayjs(booking.endTime).format("HH:mm")}
+          </div>
+        </td>
+        <td className={"flex-1 py-4 ltr:pl-4 rtl:pr-4" + (booking.rejected ? " line-through" : "")}>
+          <div className="sm:hidden">
+            {!booking.confirmed && !booking.rejected && (
+              <Tag className="mb-2 ltr:mr-2 rtl:ml-2">{t("unconfirmed")}</Tag>
             )}
-          </>
-        ) : null}
-      </td>
-    </tr>
+            {!!booking?.eventType?.price && !booking.paid && (
+              <Tag className="mb-2 ltr:mr-2 rtl:ml-2">Pending payment</Tag>
+            )}
+            <div className="text-sm font-medium text-gray-900">
+              {startTime}:{" "}
+              <small className="text-sm text-gray-500">
+                {dayjs(booking.startTime).format("HH:mm")} - {dayjs(booking.endTime).format("HH:mm")}
+              </small>
+            </div>
+          </div>
+          <div
+            title={booking.title}
+            className="max-w-56 truncate text-sm font-medium leading-6 text-neutral-900 md:max-w-max">
+            {booking.eventType?.team && <strong>{booking.eventType.team.name}: </strong>}
+            {booking.title}
+            {!!booking?.eventType?.price && !booking.paid && (
+              <Tag className="hidden ltr:ml-2 rtl:mr-2 sm:inline-flex">Pending payment</Tag>
+            )}
+            {!booking.confirmed && !booking.rejected && (
+              <Tag className="hidden ltr:ml-2 rtl:mr-2 sm:inline-flex">{t("unconfirmed")}</Tag>
+            )}
+          </div>
+          {booking.description && (
+            <div className="max-w-52 truncate text-sm text-gray-500 md:max-w-96" title={booking.description}>
+              &quot;{booking.description}&quot;
+            </div>
+          )}
+          {booking.attendees.length !== 0 && (
+            <div className="text-sm text-gray-900 hover:text-blue-500">
+              <a href={"mailto:" + booking.attendees[0].email}>{booking.attendees[0].email}</a>
+            </div>
+          )}
+        </td>
+
+        <td className="whitespace-nowrap py-4 text-right text-sm font-medium ltr:pr-4 rtl:pl-4">
+          {isUpcoming && !isCancelled ? (
+            <>
+              {!booking.confirmed && !booking.rejected && <TableActions actions={pendingActions} />}
+              {booking.confirmed && !booking.rejected && <TableActions actions={bookedActions} />}
+              {!booking.confirmed && booking.rejected && (
+                <div className="text-sm text-gray-500">{t("rejected")}</div>
+              )}
+            </>
+          ) : null}
+        </td>
+      </tr>
+    </>
   );
 }
 
diff --git a/lib/emails/templates/attendee-declined-email.ts b/lib/emails/templates/attendee-declined-email.ts
index 1665e50b..c27c3605 100644
--- a/lib/emails/templates/attendee-declined-email.ts
+++ b/lib/emails/templates/attendee-declined-email.ts
@@ -48,6 +48,7 @@ ${this.getWhat()}
 ${this.getWhen()}
 ${this.getLocation()}
 ${this.getAdditionalNotes()}
+${this.getRejectionReason()}
 `.replace(/(<([^>]+)>)/gi, "");
   }
 
@@ -95,6 +96,7 @@ ${this.getAdditionalNotes()}
                               ${this.getWho()}
                               ${this.getLocation()}
                               ${this.getAdditionalNotes()}
+                              ${this.getRejectionReason()}
                             </div>
                           </td>
                         </tr>
diff --git a/lib/emails/templates/attendee-scheduled-email.ts b/lib/emails/templates/attendee-scheduled-email.ts
index 494ed452..674a32ff 100644
--- a/lib/emails/templates/attendee-scheduled-email.ts
+++ b/lib/emails/templates/attendee-scheduled-email.ts
@@ -285,6 +285,7 @@ ${getRichDescription(this.calEvent)}
   }
 
   protected getAdditionalNotes(): string {
+    if (!this.calEvent.description) return "";
     return `
     <p style="height: 6px"></p>
     <div style="line-height: 6px;">
@@ -296,6 +297,16 @@ ${getRichDescription(this.calEvent)}
     `;
   }
 
+  protected getRejectionReason(): string {
+    if (!this.calEvent.rejectionReason) return "";
+    return `
+    <p style="height: 6px"></p>
+    <div style="line-height: 6px;">
+      <p style="color: #494949;">${this.calEvent.attendees[0].language.translate("rejection_reason")}</p>
+      <p style="color: #494949; font-weight: 400; line-height: 24px;">${this.calEvent.rejectionReason}</p>
+    </div>`;
+  }
+
   protected getLocation(): string {
     let providerName = this.calEvent.location ? getIntegrationName(this.calEvent.location) : "";
 
diff --git a/lib/integrations/calendar/interfaces/Calendar.ts b/lib/integrations/calendar/interfaces/Calendar.ts
index 0b6c1b61..c63ade15 100644
--- a/lib/integrations/calendar/interfaces/Calendar.ts
+++ b/lib/integrations/calendar/interfaces/Calendar.ts
@@ -53,6 +53,7 @@ export interface CalendarEvent {
   paymentInfo?: PaymentInfo | null;
   destinationCalendar?: DestinationCalendar | null;
   cancellationReason?: string | null;
+  rejectionReason?: string | null;
 }
 
 export interface IntegrationCalendar extends Ensure<Partial<SelectedCalendar>, "externalId"> {
diff --git a/pages/api/book/confirm.ts b/pages/api/book/confirm.ts
index a2d43fe3..9d0d62d1 100644
--- a/pages/api/book/confirm.ts
+++ b/pages/api/book/confirm.ts
@@ -3,6 +3,7 @@ import type { NextApiRequest, NextApiResponse } from "next";
 
 import { refund } from "@ee/lib/stripe/server";
 
+import { asStringOrNull } from "@lib/asStringOrNull";
 import { getSession } from "@lib/auth";
 import { sendDeclinedEmails } from "@lib/emails/email-manager";
 import { sendScheduledEmails } from "@lib/emails/email-manager";
@@ -185,7 +186,7 @@ export default async function handler(req: NextApiRequest, res: NextApiResponse)
       res.status(204).end();
     } else {
       await refund(booking, evt);
-
+      const rejectionReason = asStringOrNull(req.body.reason) || "";
       await prisma.booking.update({
         where: {
           id: bookingId,
@@ -193,6 +194,7 @@ export default async function handler(req: NextApiRequest, res: NextApiResponse)
         data: {
           rejected: true,
           status: BookingStatus.REJECTED,
+          rejectionReason: rejectionReason,
         },
       });
 
diff --git a/prisma/migrations/20220209082843_add_rejection_reason/migration.sql b/prisma/migrations/20220209082843_add_rejection_reason/migration.sql
new file mode 100644
index 00000000..aed38fc2
--- /dev/null
+++ b/prisma/migrations/20220209082843_add_rejection_reason/migration.sql
@@ -0,0 +1,2 @@
+-- AlterTable
+ALTER TABLE "Booking" ADD COLUMN     "rejectionReason" TEXT;
\ No newline at end of file
diff --git a/prisma/schema.prisma b/prisma/schema.prisma
index bbc591f9..89a0dbcc 100644
--- a/prisma/schema.prisma
+++ b/prisma/schema.prisma
@@ -245,6 +245,7 @@ model Booking {
   payment             Payment[]
   destinationCalendar DestinationCalendar?
   cancellationReason  String?
+  rejectionReason  String?
 }
 
 model Schedule {
diff --git a/prisma/zod/booking.ts b/prisma/zod/booking.ts
index 4402bb48..1e618895 100644
--- a/prisma/zod/booking.ts
+++ b/prisma/zod/booking.ts
@@ -36,6 +36,7 @@ export const _BookingModel = z.object({
   status: z.nativeEnum(BookingStatus),
   paid: z.boolean(),
   cancellationReason: z.string().nullish(),
+  rejectionReason: z.string().nullish(),
 });
 
 export interface CompleteBooking extends z.infer<typeof _BookingModel> {
diff --git a/public/static/locales/en/common.json b/public/static/locales/en/common.json
index 4fec5d30..8711346e 100644
--- a/public/static/locales/en/common.json
+++ b/public/static/locales/en/common.json
@@ -15,6 +15,10 @@
   "need_to_reschedule_or_cancel": "Need to reschedule or cancel?",
   "cancellation_reason": "Reason for cancellation",
   "cancellation_reason_placeholder": "Why are you cancelling? (optional)",
+  "rejection_reason": "Reason for rejecting",
+  "rejection_reason_title": "Reject the booking request?",
+  "rejection_reason_description": "Are you sure you want to reject the booking? We'll let the person who tried to book know. You can provide a reason below.",
+  "rejection_confirmation": "Reject the booking",
   "manage_this_event": "Manage this event",
   "your_event_has_been_scheduled": "Your event has been scheduled",
   "accept_our_license": "Accept our license by changing the .env variable <1>NEXT_PUBLIC_LICENSE_CONSENT</1> to '{{agree}}'.",