diff --git a/apps/sim/blocks/blocks/sap_concur.ts b/apps/sim/blocks/blocks/sap_concur.ts index b67e2178f4..f7e83b6bfb 100644 --- a/apps/sim/blocks/blocks/sap_concur.ts +++ b/apps/sim/blocks/blocks/sap_concur.ts @@ -44,6 +44,7 @@ const EXPENSE_READ_CONTEXT_TYPE_OPS = [ 'sap_concur_list_expenses', 'sap_concur_get_expense', 'sap_concur_get_itemizations', + 'sap_concur_create_report_comment', 'sap_concur_list_report_comments', 'sap_concur_list_exceptions', ] @@ -59,7 +60,6 @@ const ATTENDEE_CONTEXT_TYPE_OPS = [ 'sap_concur_list_attendee_associations', 'sap_concur_associate_attendees', 'sap_concur_remove_all_attendees', - 'sap_concur_create_report_comment', ] const ALLOCATION_CONTEXT_TYPE_OPS = [ @@ -928,7 +928,7 @@ export const SapConcurBlock: BlockConfig = { placeholder: 'List ID', condition: { field: 'operation', - value: ['sap_concur_get_list', 'sap_concur_list_list_items', 'sap_concur_get_list_item'], + value: ['sap_concur_get_list', 'sap_concur_list_list_items'], }, required: { field: 'operation', @@ -1080,7 +1080,7 @@ export const SapConcurBlock: BlockConfig = { id: 'lastModifiedDate', title: 'Last Modified Date', type: 'short-input', - placeholder: 'YYYY-MM-DD or 1900-01-01T12:00:00', + placeholder: '1900-01-01T00:00:00 (UTC datetime)', condition: { field: 'operation', value: 'sap_concur_list_travel_profiles_summary', @@ -1110,8 +1110,8 @@ export const SapConcurBlock: BlockConfig = { type: 'dropdown', options: [ { label: 'Any', id: '' }, - { label: 'Active', id: 'Active' }, - { label: 'Inactive', id: 'Inactive' }, + { label: 'Active', id: '1' }, + { label: 'Inactive', id: '0' }, ], value: () => '', condition: { field: 'operation', value: 'sap_concur_list_travel_profiles_summary' }, @@ -1684,7 +1684,6 @@ export const SapConcurBlock: BlockConfig = { case 'sap_concur_get_list_item': return { ...auth, - listId: params.listId || undefined, itemId: params.itemId, } case 'sap_concur_list_budgets': @@ -1723,7 +1722,7 @@ export const SapConcurBlock: BlockConfig = { page: params.travelProfilePage ? Number(params.travelProfilePage) : undefined, itemsPerPage: params.itemsPerPage ? Number(params.itemsPerPage) : undefined, active: - params.travelProfileActive === 'Active' || params.travelProfileActive === 'Inactive' + params.travelProfileActive === '1' || params.travelProfileActive === '0' ? params.travelProfileActive : undefined, travelConfigs: params.travelConfigs || undefined, @@ -1886,7 +1885,10 @@ export const SapConcurBlock: BlockConfig = { page: { type: 'number', description: 'Page number (lists/list_items)' }, travelProfilePage: { type: 'number', description: 'Profile summary page number' }, itemsPerPage: { type: 'number', description: 'Profile summary items per page' }, - travelProfileActive: { type: 'string', description: 'Status filter ("Active" or "Inactive")' }, + travelProfileActive: { + type: 'string', + description: 'Active filter ("1" for active, "0" for inactive)', + }, travelConfigs: { type: 'string', description: 'Comma-separated travel config ids' }, searchText: { type: 'string', description: 'Locations v5 free-text search' }, locCode: { type: 'string', description: 'Locations v5 location code' }, diff --git a/apps/sim/tools/sap_concur/associate_attendees.ts b/apps/sim/tools/sap_concur/associate_attendees.ts index 2efd05707c..ae75a711c1 100644 --- a/apps/sim/tools/sap_concur/associate_attendees.ts +++ b/apps/sim/tools/sap_concur/associate_attendees.ts @@ -12,7 +12,7 @@ export const associateAttendeesTool: ToolConfig { const tripId = trimRequired(params.tripId, 'tripId') const query = buildListQuery({ - useridType: params.useridType, - useridValue: params.useridValue, + userid_type: params.useridType, + userid_value: params.useridValue, systemFormat: params.systemFormat, }) return { diff --git a/apps/sim/tools/sap_concur/list_expenses.ts b/apps/sim/tools/sap_concur/list_expenses.ts index 744973df63..8c90cd7749 100644 --- a/apps/sim/tools/sap_concur/list_expenses.ts +++ b/apps/sim/tools/sap_concur/list_expenses.ts @@ -66,7 +66,7 @@ export const listExpensesTool: ToolConfig