From 33e81987796db2dc3057d927f54e8f61788875ce Mon Sep 17 00:00:00 2001
From: Joe Au-Yeung <65426560+joeauyeung@users.noreply.github.com>
Date: Sat, 17 Sep 2022 16:44:29 -0400
Subject: [PATCH] Hotfix To Seats on Event Types (#4528)

* Hotfix

* Add seats input to event type page
---
 .../components/booking/pages/BookingPage.tsx  | 10 ++---
 .../v2/eventtype/EventAdvancedTab.tsx         | 42 +++++++++++++++++++
 apps/web/pages/api/book/event.ts              | 17 ++++----
 apps/web/public/static/locales/en/common.json |  5 ++-
 4 files changed, 58 insertions(+), 16 deletions(-)

diff --git a/apps/web/components/booking/pages/BookingPage.tsx b/apps/web/components/booking/pages/BookingPage.tsx
index 8067a6f8d7..b89182af17 100644
--- a/apps/web/components/booking/pages/BookingPage.tsx
+++ b/apps/web/components/booking/pages/BookingPage.tsx
@@ -199,16 +199,12 @@ const BookingPage = ({
 
   // There should only exists one default userData variable for primaryAttendee.
   const defaultUserValues = {
-    email: booking?.attendees[0].email
-      ? booking.attendees[0].email
+    email: rescheduleUid
+      ? booking?.attendees[0].email
       : router.query.email
       ? (router.query.email as string)
       : "",
-    name: booking?.attendees[0].name
-      ? booking.attendees[0].name
-      : router.query.name
-      ? (router.query.name as string)
-      : "",
+    name: rescheduleUid ? booking?.attendees[0].name : router.query.name ? (router.query.name as string) : "",
   };
 
   const defaultValues = () => {
diff --git a/apps/web/components/v2/eventtype/EventAdvancedTab.tsx b/apps/web/components/v2/eventtype/EventAdvancedTab.tsx
index c307f28ce0..17f1403c0c 100644
--- a/apps/web/components/v2/eventtype/EventAdvancedTab.tsx
+++ b/apps/web/components/v2/eventtype/EventAdvancedTab.tsx
@@ -41,6 +41,8 @@ export const EventAdvancedTab = ({ eventType, team }: Pick<EventTypeSetupInfered
   const [showEventNameTip, setShowEventNameTip] = useState(false);
   const [hashedLinkVisible, setHashedLinkVisible] = useState(!!eventType.hashedLink);
   const [hashedUrl, setHashedUrl] = useState(eventType.hashedLink?.link);
+  const [seatsInputVisible, setSeatsInputVisible] = useState(!!eventType.seatsPerTimeSlot);
+  const [seatsPerTimeSlot, setSeatsPerTimeSlot] = useState(eventType.seatsPerTimeSlot);
   const [customInputs, setCustomInputs] = useState<EventTypeCustomInput[]>(
     eventType.customInputs.sort((a, b) => a.id - b.id) || []
   );
@@ -293,6 +295,46 @@ export const EventAdvancedTab = ({ eventType, team }: Pick<EventTypeSetupInfered
           </>
         )}
       />
+      <hr />
+      <Controller
+        name="seatsPerTimeSlot"
+        control={formMethods.control}
+        defaultValue={seatsPerTimeSlot}
+        render={({ field: { value, onChange } }) => (
+          <>
+            <div className="flex space-x-3">
+              <Switch
+                name="seatsPerTimeSlot"
+                checked={seatsInputVisible}
+                onCheckedChange={(e) => {
+                  setSeatsInputVisible(e);
+                  onChange(e ? seatsPerTimeSlot : null);
+                }}
+                fitToHeight={true}
+              />
+              <div className="flex flex-col">
+                <Label className="text-sm font-semibold leading-none text-black">{t("offer_seats")}</Label>
+                <p className="-mt-2 text-sm leading-normal text-gray-600">{t("offer_seats_description")}</p>
+              </div>
+            </div>
+            {seatsInputVisible && (
+              <div className="">
+                <TextField
+                  required
+                  name="seatsPerTimeSlot"
+                  label={t("number_of_seats")}
+                  type="number"
+                  defaultValue={seatsPerTimeSlot || 2}
+                  addOnSuffix={<>{t("seats")}</>}
+                  onChange={(e) => {
+                    formMethods.setValue("seatsPerTimeSlot", Number(e.target.value));
+                  }}
+                />
+              </div>
+            )}
+          </>
+        )}
+      />
       {showEventNameTip && (
         <Dialog open={showEventNameTip} onOpenChange={setShowEventNameTip}>
           <DialogContent
diff --git a/apps/web/pages/api/book/event.ts b/apps/web/pages/api/book/event.ts
index aa3eee55bd..f5fc833cff 100644
--- a/apps/web/pages/api/book/event.ts
+++ b/apps/web/pages/api/book/event.ts
@@ -338,15 +338,18 @@ async function handler(req: NextApiRequest) {
     }
   );
 
+  // Add an if conditional if there are no seats on the event type
   // Assign to only one user when ROUND_ROBIN
-  if (eventType.schedulingType === SchedulingType.ROUND_ROBIN) {
-    users = [await getLuckyUser("MAXIMIZE_AVAILABILITY", { availableUsers, eventTypeId: eventType.id })];
-  } else {
-    // excluding ROUND_ROBIN, all users have availability required.
-    if (availableUsers.length !== users.length) {
-      throw new Error("Some users are unavailable for booking.");
+  if (!eventType.seatsPerTimeSlot) {
+    if (eventType.schedulingType === SchedulingType.ROUND_ROBIN) {
+      users = [await getLuckyUser("MAXIMIZE_AVAILABILITY", { availableUsers, eventTypeId: eventType.id })];
+    } else {
+      // excluding ROUND_ROBIN, all users have availability required.
+      if (availableUsers.length !== users.length) {
+        throw new Error("Some users are unavailable for booking.");
+      }
+      users = availableUsers;
     }
-    users = availableUsers;
   }
 
   console.log("available users", users);
diff --git a/apps/web/public/static/locales/en/common.json b/apps/web/public/static/locales/en/common.json
index 64a8468306..dfc6c13c1d 100644
--- a/apps/web/public/static/locales/en/common.json
+++ b/apps/web/public/static/locales/en/common.json
@@ -843,7 +843,7 @@
   "redirect_url_upgrade_description": "In order to use this feature, you need to upgrade to a Pro account.",
   "duplicate": "Duplicate",
   "offer_seats": "Offer seats",
-  "offer_seats_description": "Offer seats to bookings (This disables guests & opt in bookings)",
+  "offer_seats_description": "Offer seats for booking. This automatically disables guest & opt-in bookings.",
   "seats_available": "Seats available",
   "number_of_seats": "Number of seats per booking",
   "enter_number_of_seats": "Enter number of seats",
@@ -1265,5 +1265,6 @@
   "no_calendar_installed": "No calendar installed",
   "no_calendar_installed_description": "You have not yet connected any of your calendars",
   "add_a_calendar": "Add a calendar",
-  "change_email_hint": "You may need to log out and back in to see any change take effect"
+  "change_email_hint": "You may need to log out and back in to see any change take effect",
+  "seats": "seats"
 }