fix: my units calendar now shows the reservation name with prefix

- makes translation function a required parameter in getReserveeName
City-of-Helsinki · May 23, 2024 · c46157f · c46157f
1 parent b9db783
commit c46157f
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 5 deletions.
diff --git a/apps/admin-ui/src/component/my-units/ReservationUnitCalendar.tsx b/apps/admin-ui/src/component/my-units/ReservationUnitCalendar.tsx
@@ -28,6 +28,7 @@ import {
 } from "common/src/helpers";
 import { RELATED_RESERVATION_STATES } from "common/src/const";
 import { ReservationUnitWithAffectingArgs } from "common/src/queries/fragments";
+import { TFunction } from "next-i18next";
 
 type Props = {
   begin: string;
@@ -50,14 +51,16 @@ const Container = styled.div`
 const getEventTitle = ({
   reservationUnitPk,
   reservation,
+  t,
 }: {
   reservationUnitPk: number;
   reservation: ReservationNode;
+  t: TFunction;
 }) => {
   const reservationUnit = reservation.reservationUnit?.[0];
   const isOtherReservationUnit = reservationUnitPk !== reservationUnit?.pk;
 
-  const reserveeName = getReserveeName(reservation);
+  const reserveeName = getReserveeName(reservation, t);
   if (isOtherReservationUnit) {
     const reservationUnitName = reservationUnit?.nameFi ?? "";
 
@@ -67,10 +70,15 @@ const getEventTitle = ({
   return [reserveeName, ""];
 };
 
-const constructEventTitle = (res: ReservationNode, resUnitPk: number) => {
+const constructEventTitle = (
+  res: ReservationNode,
+  resUnitPk: number,
+  t: TFunction
+) => {
   const [reservee, unit] = getEventTitle({
     reservationUnitPk: resUnitPk,
     reservation: res,
+    t,
   });
   if (unit.length > 0) {
     return `${reservee} (${unit})`;
@@ -124,7 +132,7 @@ export function ReservationUnitCalendar({
   const events = reservations.map((reservation) => {
     const isBlocked = reservation.type === ReservationTypeChoice.Blocked;
     const title = !isBlocked
-      ? constructEventTitle(reservation, reservationUnitPk)
+      ? constructEventTitle(reservation, reservationUnitPk, t)
       : t("MyUnits.Calendar.legend.closed");
     return {
       title,

diff --git a/apps/admin-ui/src/component/my-units/UnitCalendar.tsx b/apps/admin-ui/src/component/my-units/UnitCalendar.tsx
@@ -317,7 +317,7 @@ function getEventTitle({
   }
 
   return event && event?.pk !== event?.reservationUnit?.[0]?.pk
-    ? getReserveeName(event)
+    ? getReserveeName(event, t)
     : title;
 }
 

diff --git a/apps/admin-ui/src/component/reservations/requested/util.ts b/apps/admin-ui/src/component/reservations/requested/util.ts
@@ -198,7 +198,7 @@ export const getTranslationKeyForCustomerTypeChoice = (
 
 export const getReserveeName = (
   reservation: ReservationNode,
-  t?: TFunction,
+  t: TFunction,
   length = 50
 ): string => {
   let prefix = "";