admin/n8n-enterprise-unlocked
1
0
Fork 0
mirror of https://github.com/Abdulazizzn/n8n-enterprise-unlocked.git synced 2025-12-22 04:10:01 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(Google Sheets Node): Append exceeding grid limits (#7684)

Browse Source 
Github issue / Community forum post (link here to close automatically):
https://community.n8n.io/t/error-appending-data-with-google-sheets/28691
This commit is contained in:
Michael Kret
2023-11-15 11:33:47 +02:00
committed by GitHub
parent 4441ed5116
commit 88efb99587
3 changed files with 85 additions and 44 deletions
Download Patch File Download Diff File Expand all files Collapse all files

53
packages/nodes-base/nodes/Google/Sheet/v2/helpers/GoogleSheet.ts
View File

@@ -247,24 +247,8 @@ export class GoogleSheet {
data: string[][],
valueInputMode: ValueInputOption,
lastRow?: number,
useAppend?: boolean,
) {
// const body = {
// range,
// values: data,
// };
// const query = {
// valueInputOption: valueInputMode,
// };
// const response = await apiRequest.call(
// this.executeFunctions,
// 'POST',
// `/v4/spreadsheets/${this.id}/values/${this.encodeRange(range)}:append`,
// body,
// query,
// );
const lastRowWithData =
lastRow ||
(((await this.getData(range, 'UNFORMATTED_VALUE')) as string[][]) || []).length + 1;
@@ -275,6 +259,7 @@ export class GoogleSheet {
valueInputMode,
lastRowWithData,
data.length,
useAppend,
);
return response;
@@ -286,6 +271,7 @@ export class GoogleSheet {
valueInputMode: ValueInputOption,
row: number,
rowsLength?: number,
useAppend?: boolean,
) {
const [name, _sheetRange] = sheetName.split('!');
const range = `${name}!${row}:${rowsLength ? row + rowsLength - 1 : row}`;
@@ -299,13 +285,25 @@ export class GoogleSheet {
valueInputOption: valueInputMode,
};
const response = await apiRequest.call(
this.executeFunctions,
'PUT',
`/v4/spreadsheets/${this.id}/values/${this.encodeRange(range)}`,
body,
query,
);
let response;
if (useAppend) {
response = await apiRequest.call(
this.executeFunctions,
'POST',
`/v4/spreadsheets/${this.id}/values/${this.encodeRange(range)}:append`,
body,
query,
);
} else {
response = await apiRequest.call(
this.executeFunctions,
'PUT',
`/v4/spreadsheets/${this.id}/values/${this.encodeRange(range)}`,
body,
query,
);
}
return response;
}
@@ -386,6 +384,7 @@ export class GoogleSheet {
usePathForKeyRow: boolean,
columnNamesList?: string[][],
lastRow?: number,
useAppend?: boolean,
): Promise<string[][]> {
const data = await this.convertObjectArrayToSheetDataArray(
inputData,
@@ -393,8 +392,9 @@ export class GoogleSheet {
keyRowIndex,
usePathForKeyRow,
columnNamesList,
useAppend ? null : '',
);
return this.appendData(range, data, valueInputMode, lastRow);
return this.appendData(range, data, valueInputMode, lastRow, useAppend);
}
getColumnWithOffset(startColumn: string, offset: number): string {
@@ -700,6 +700,7 @@ export class GoogleSheet {
keyRowIndex: number,
usePathForKeyRow: boolean,
columnNamesList?: string[][],
emptyValue: string | null = '',
): Promise<string[][]> {
const decodedRange = this.getDecodedSheetRange(range);
@@ -730,7 +731,7 @@ export class GoogleSheet {
value = item[key] as string;
}
if (value === undefined || value === null) {
rowData.push('');
rowData.push(emptyValue as string);
return;
}
if (typeof value === 'object') {