fix: use existing isTeamAdmin function and remove unnecessary UI changes

Author: neha

apps/web/modules/bookings/views/bookings-single-view.getServerSideProps.tsx

@@ -11,6 +11,7 @@ import { shouldHideBrandingForEvent } from "@calcom/lib/hideBranding";
 import { parseRecurringEvent } from "@calcom/lib/isRecurringEvent";
 import { markdownToSafeHTML } from "@calcom/lib/markdownToSafeHTML";
 import { maybeGetBookingUidFromSeat } from "@calcom/lib/server/maybeGetBookingUidFromSeat";
+import { isTeamAdmin } from "@calcom/lib/server/queries/teams";
 import { BookingRepository } from "@calcom/lib/server/repository/booking";
 import prisma from "@calcom/prisma";
 import { customInputSchema } from "@calcom/prisma/zod-utils";
@@ -178,22 +179,8 @@ export async function getServerSideProps(context: GetServerSidePropsContext) {
 
   const isLoggedInUserTeamAdmin = !!(
     userId &&
-    ((eventType.team?.id &&
-      (await prisma.membership.findUnique({
-        where: {
-          userId_teamId: { userId, teamId: eventType.team.id },
-          accepted: true,
-          OR: [{ role: "ADMIN" }, { role: "OWNER" }],
-        },
-      }))) ||
-      (eventType.parent?.teamId &&
-        (await prisma.membership.findUnique({
-          where: {
-            userId_teamId: { userId, teamId: eventType.parent.teamId },
-            accepted: true,
-            OR: [{ role: "ADMIN" }, { role: "OWNER" }],
-          },
-        }))))
+    ((eventType.team?.id && (await isTeamAdmin(userId, eventType.team.id))) ||
+      (eventType.parent?.teamId && (await isTeamAdmin(userId, eventType.parent.teamId))))
   );
 
   const canViewHiddenData = isLoggedInUserHost || isLoggedInUserTeamAdmin;

apps/web/modules/bookings/views/bookings-single-view.tsx

@@ -385,7 +385,7 @@ export default function Success(props: PageProps) {
   const isRescheduled = bookingInfo?.rescheduled;
 
   const canCancelOrReschedule = !eventType?.disableCancelling || !eventType?.disableRescheduling;
-  const canCancelAndReschedule = !eventType?.disableCancelling && !eventType?.disableRescheduling;
+  const _canCancelAndReschedule = !eventType?.disableCancelling && !eventType?.disableRescheduling;
 
   const canCancel = !eventType?.disableCancelling;
   const canReschedule = !eventType?.disableRescheduling;
@@ -471,7 +471,7 @@ export default function Success(props: PageProps) {
                 {!isFeedbackMode && (
                   <>
                     <div
-                      className={classNames(isRoundRobin && "relative mx-auto h-24 min-h-24 w-32 min-w-32")}>
+                      className={classNames(isRoundRobin && "min-w-22 w-22 relative mx-auto h-24 min-h-24")}>
                       {isRoundRobin && bookingInfo.user && (
                         <Avatar
                           className="mx-auto flex items-center justify-center"
@@ -557,7 +557,7 @@ export default function Success(props: PageProps) {
                           </h4>
                         )}
 
-                      <div className="border-subtle text-default mt-8 grid grid-cols-3 gap-x-4 border-t pt-8 text-left sm:gap-x-0 rtl:text-right">
+                      <div className="border-subtle text-default mt-2 grid grid-cols-2 gap-x-4 border-t pt-2 text-left rtl:text-right">
                         {(isCancelled || reschedule) && cancellationReason && (
                           <>
                             <div className="font-medium">
@@ -1081,7 +1081,7 @@ export default function Success(props: PageProps) {
               </div>
               {isGmail && !isFeedbackMode && (
                 <Alert
-                  className="main -mb-20 mt-4 inline-block sm:-mt-4 sm:mb-4 sm:w-full sm:max-w-xl sm:align-middle ltr:text-left rtl:text-right"
+                  className="main -mb-20 mt-4 inline-block ltr:text-left rtl:text-right"
                   severity="warning"
                   message={
                     <div>