refactor: enhance type handling in acknowledgeOooRequest

- Updated requestData type assertion to include oldOooStatusRequest for better type safety.
- Removed legacy request format normalization logic to streamline the function and improve clarity.

Author: RishiChaubey31

services/oooRequest.ts

@@ -147,18 +147,12 @@ export const acknowledgeOooRequest = async (
     superUserId: string,
 ) => {
     try{
-        const requestData = await getRequests({ id: requestId });
+        const requestData = await getRequests({ id: requestId }) as OooStatusRequest | oldOooStatusRequest;
         if (!requestData) {
             throw new NotFound("Request not found");
         }
-        //TODO: Remove this after old request format is removed
-        const normalized: OooStatusRequest = (
-            (requestData as OooStatusRequest).type === REQUEST_TYPE.OOO &&
-            'state' in (requestData as OooStatusRequest) &&
-            !('status' in (requestData as OooStatusRequest))
-        ) ? newOOOSchema(requestData as oldOooStatusRequest) as OooStatusRequest : requestData as OooStatusRequest;
+    const { type, status, from, until, requestedBy } = requestData as OooStatusRequest;
 
-        const { type, status, from, until, requestedBy } = normalized;
         await validateOooAcknowledgeRequest(type as string, status as string);
         const requestResult = await updateRequest(requestId, body, superUserId, REQUEST_TYPE.OOO);
         if(requestResult.error){