Skip to content

Commit 4c6c9e8

Browse files
committed
fix(selectors): rename pre-existing driveId and files canonicalParamIds in SharePoint
Apply the same selected_ prefix convention to the pre-existing SharePoint driveId and files canonical params that clashed with their subBlock IDs.
1 parent 8512dbd commit 4c6c9e8

File tree

1 file changed

+9
-7
lines changed

1 file changed

+9
-7
lines changed

apps/sim/blocks/blocks/sharepoint.ts

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -348,7 +348,7 @@ Return ONLY the JSON object - no explanations, no markdown, no extra text.`,
348348
title: 'Document Library ID',
349349
type: 'short-input',
350350
placeholder: 'Enter document library (drive) ID',
351-
canonicalParamId: 'driveId',
351+
canonicalParamId: 'selected_driveId',
352352
condition: { field: 'operation', value: 'upload_file' },
353353
mode: 'advanced',
354354
},
@@ -374,7 +374,7 @@ Return ONLY the JSON object - no explanations, no markdown, no extra text.`,
374374
id: 'uploadFiles',
375375
title: 'Files',
376376
type: 'file-upload',
377-
canonicalParamId: 'files',
377+
canonicalParamId: 'selected_files',
378378
placeholder: 'Upload files to SharePoint',
379379
condition: { field: 'operation', value: 'upload_file' },
380380
mode: 'basic',
@@ -386,7 +386,7 @@ Return ONLY the JSON object - no explanations, no markdown, no extra text.`,
386386
id: 'files',
387387
title: 'Files',
388388
type: 'short-input',
389-
canonicalParamId: 'files',
389+
canonicalParamId: 'selected_files',
390390
placeholder: 'Reference files from previous blocks',
391391
condition: { field: 'operation', value: 'upload_file' },
392392
mode: 'advanced',
@@ -438,7 +438,8 @@ Return ONLY the JSON object - no explanations, no markdown, no extra text.`,
438438
listItemFields, // canonical param
439439
includeColumns,
440440
includeItems,
441-
files, // canonical param from uploadFiles (basic) or files (advanced)
441+
selected_files, // canonical param from uploadFiles (basic) or files (advanced)
442+
selected_driveId, // canonical param from driveId
442443
columnDefinitions,
443444
selected_listId,
444445
...others
@@ -485,14 +486,15 @@ Return ONLY the JSON object - no explanations, no markdown, no extra text.`,
485486
}
486487

487488
// Handle file upload files parameter using canonical param
488-
const normalizedFiles = normalizeFileInput(files)
489+
const normalizedFiles = normalizeFileInput(selected_files)
489490
const baseParams: Record<string, any> = {
490491
oauthCredential,
491492
siteId: effectiveSiteId || undefined,
492493
pageSize: others.pageSize ? Number.parseInt(others.pageSize as string, 10) : undefined,
493494
mimeType: mimeType,
494495
...others,
495496
...(selected_listId ? { listId: selected_listId } : {}),
497+
...(selected_driveId ? { driveId: selected_driveId } : {}),
496498
itemId: sanitizedItemId,
497499
listItemFields: parsedItemFields,
498500
includeColumns: coerceBoolean(includeColumns),
@@ -533,10 +535,10 @@ Return ONLY the JSON object - no explanations, no markdown, no extra text.`,
533535
includeItems: { type: 'boolean', description: 'Include items in response' },
534536
itemId: { type: 'string', description: 'List item ID (canonical param)' },
535537
listItemFields: { type: 'string', description: 'List item fields (canonical param)' },
536-
driveId: { type: 'string', description: 'Document library (drive) ID (canonical param)' },
538+
selected_driveId: { type: 'string', description: 'Document library (drive) ID (canonical param)' },
537539
folderPath: { type: 'string', description: 'Folder path for file upload' },
538540
fileName: { type: 'string', description: 'File name override' },
539-
files: { type: 'array', description: 'Files to upload (canonical param)' },
541+
selected_files: { type: 'array', description: 'Files to upload (canonical param)' },
540542
},
541543
outputs: {
542544
sites: {

0 commit comments

Comments
 (0)