address comments

Author: icecrasher321

apps/sim/app/api/tools/workday/create-prehire/route.ts

@@ -32,6 +32,16 @@ export async function POST(request: NextRequest) {
     const body = await request.json()
     const data = RequestSchema.parse(body)
 
+    if (!data.email && !data.phoneNumber && !data.address) {
+      return NextResponse.json(
+        {
+          success: false,
+          error: 'At least one contact method (email, phone, or address) is required',
+        },
+        { status: 400 }
+      )
+    }
+
     const parts = data.legalName.trim().split(/\s+/)
     const firstName = parts[0] ?? ''
     const lastName = parts.length > 1 ? parts.slice(1).join(' ') : (parts[0] ?? '')
@@ -44,16 +54,6 @@ export async function POST(request: NextRequest) {
       data.password
     )
 
-    if (!data.email && !data.phoneNumber && !data.address) {
-      return NextResponse.json(
-        {
-          success: false,
-          error: 'At least one contact method (email, phone, or address) is required',
-        },
-        { status: 400 }
-      )
-    }
-
     const contactData: Record<string, unknown> = {}
     if (data.email) {
       contactData.Email_Address_Data = [

apps/sim/blocks/blocks/workday.ts

@@ -349,18 +349,27 @@ Output: {"businessTitle": "Senior Engineer"}`,
     config: {
       tool: (params) => `workday_${params.operation}`,
       params: (params) => {
-        const { operation, orgType, fields, ...rest } = params
+        const {
+          operation,
+          orgType,
+          fields,
+          positionId,
+          jobProfileId,
+          locationId,
+          managerId,
+          ...rest
+        } = params
 
         if (rest.limit != null && rest.limit !== '') rest.limit = Number(rest.limit)
         if (rest.offset != null && rest.offset !== '') rest.offset = Number(rest.offset)
 
         if (orgType) rest.type = orgType
 
         if (operation === 'change_job') {
-          if (rest.positionId) rest.newPositionId = rest.positionId
-          if (rest.jobProfileId) rest.newJobProfileId = rest.jobProfileId
-          if (rest.locationId) rest.newLocationId = rest.locationId
-          if (rest.managerId) rest.newManagerId = rest.managerId
+          if (positionId) rest.newPositionId = positionId
+          if (jobProfileId) rest.newJobProfileId = jobProfileId
+          if (locationId) rest.newLocationId = locationId
+          if (managerId) rest.newManagerId = managerId
         }
 
         if (fields && operation === 'update_worker') {