Skip to content

Commit 8296716

Browse files
kevinkim-ogpkevinkim-ogp
authored
Release v1.53.3 (#1232)
* chore: update message when form is not public * fix: for each not showing some excel columns
2 parents 644ce30 + 4a11cc1 commit 8296716

File tree

8 files changed

+200
-86
lines changed

8 files changed

+200
-86
lines changed

‎package-lock.json‎

Lines changed: 3 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎packages/backend/package.json‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -111,5 +111,5 @@
111111
"tsconfig-paths": "^4.2.0",
112112
"type-fest": "4.10.3"
113113
},
114-
"version": "1.53.2"
114+
"version": "1.53.3"
115115
}

‎packages/backend/src/apps/toolbox/__tests__/actions/for-each/get-data-out-metadata.test.ts‎

Lines changed: 160 additions & 76 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,19 @@ import { describe, expect, it } from 'vitest'
33
import getDataOutMetadata from '../../../actions/for-each/get-data-out-metadata'
44
import { FOR_EACH_INPUT_SOURCE } from '../../../common/constants'
55

6+
const GENERIC_TABLE_EXPECTED_RESULT = {
7+
iteration: {
8+
label: 'Item number',
9+
displayedValue: '1',
10+
},
11+
iterations: { label: 'Items found' },
12+
inputSource: { isHidden: true },
13+
items: {
14+
rows: [{ data: { isHidden: true }, rowId: { isHidden: true } }],
15+
inputSource: { isHidden: true },
16+
},
17+
}
18+
619
describe('getDataOutMetadata', () => {
720
const createMockExecutionStep = (dataOut: any) => ({
821
id: 'execution-step-id',
@@ -199,16 +212,7 @@ describe('getDataOutMetadata', () => {
199212
const result = await getDataOutMetadata(executionStep)
200213

201214
expect(result).toEqual({
202-
iteration: {
203-
label: 'Item number',
204-
displayedValue: '1',
205-
},
206-
iterations: {
207-
label: 'Items found',
208-
},
209-
inputSource: {
210-
isHidden: true,
211-
},
215+
...GENERIC_TABLE_EXPECTED_RESULT,
212216
items: {
213217
columns: [
214218
{
@@ -268,16 +272,8 @@ describe('getDataOutMetadata', () => {
268272
const result = await getDataOutMetadata(executionStep)
269273

270274
expect(result).toEqual({
271-
iteration: {
272-
label: 'Item number',
273-
displayedValue: '',
274-
},
275-
iterations: {
276-
label: 'Items found',
277-
},
278-
inputSource: {
279-
isHidden: true,
280-
},
275+
...GENERIC_TABLE_EXPECTED_RESULT,
276+
iteration: { label: 'Item number', displayedValue: '' },
281277
items: {
282278
columns: [
283279
{
@@ -318,16 +314,7 @@ describe('getDataOutMetadata', () => {
318314
const result = await getDataOutMetadata(executionStep)
319315

320316
expect(result).toEqual({
321-
iteration: {
322-
label: 'Item number',
323-
displayedValue: '1',
324-
},
325-
iterations: {
326-
label: 'Items found',
327-
},
328-
inputSource: {
329-
isHidden: true,
330-
},
317+
...GENERIC_TABLE_EXPECTED_RESULT,
331318
items: {
332319
columns: [
333320
{
@@ -351,6 +338,144 @@ describe('getDataOutMetadata', () => {
351338
},
352339
})
353340
})
341+
342+
it.each(['Recruiter', 'sixty', 'itty'])(
343+
'should not hide column when the hex encoded column happens to be a number: %s',
344+
async (columnName) => {
345+
const testHexColumnId = Buffer.from(columnName).toString('hex')
346+
const dataOut = {
347+
iterations: 1,
348+
inputSource: FOR_EACH_INPUT_SOURCE.M365_EXCEL,
349+
items: {
350+
columns: {
351+
[testHexColumnId]: {
352+
id: testHexColumnId,
353+
name: columnName,
354+
value: `items.rows.__ITERATION__.data.${testHexColumnId}`,
355+
order: 1,
356+
},
357+
},
358+
rows: [{ data: { [testHexColumnId]: 'Vaser' } }],
359+
inputSource: FOR_EACH_INPUT_SOURCE.M365_EXCEL,
360+
},
361+
}
362+
const executionStep = createMockExecutionStep(dataOut)
363+
const result = await getDataOutMetadata(executionStep)
364+
365+
expect(result).toEqual({
366+
...GENERIC_TABLE_EXPECTED_RESULT,
367+
items: {
368+
...GENERIC_TABLE_EXPECTED_RESULT.items,
369+
columns: {
370+
[testHexColumnId]: {
371+
id: { isHidden: true },
372+
name: { isHidden: true },
373+
order: { isHidden: true },
374+
value: {
375+
label: columnName,
376+
displayedValue: 'Vaser',
377+
order: 1,
378+
type: 'text',
379+
isHiddenFromList: false,
380+
},
381+
},
382+
},
383+
},
384+
})
385+
},
386+
)
387+
388+
it.each(['Date Sent', 'Send Status'])(
389+
'should not hide columns when the hex encoded column name contains an e and appears like scientific notation: %s',
390+
async (columnName) => {
391+
const testHexColumnId = Buffer.from(columnName).toString('hex')
392+
const dataOut = {
393+
iterations: 1,
394+
inputSource: FOR_EACH_INPUT_SOURCE.M365_EXCEL,
395+
items: {
396+
columns: {
397+
[testHexColumnId]: {
398+
id: testHexColumnId,
399+
name: columnName,
400+
value: `items.rows.__ITERATION__.data.${testHexColumnId}`,
401+
order: 1,
402+
},
403+
},
404+
rows: [{ data: { [testHexColumnId]: 'Vaser' } }],
405+
inputSource: FOR_EACH_INPUT_SOURCE.M365_EXCEL,
406+
},
407+
}
408+
const executionStep = createMockExecutionStep(dataOut)
409+
const result = await getDataOutMetadata(executionStep)
410+
411+
expect(result).toEqual({
412+
...GENERIC_TABLE_EXPECTED_RESULT,
413+
items: {
414+
...GENERIC_TABLE_EXPECTED_RESULT.items,
415+
columns: {
416+
[testHexColumnId]: {
417+
id: { isHidden: true },
418+
name: { isHidden: true },
419+
order: { isHidden: true },
420+
value: {
421+
label: columnName,
422+
displayedValue: 'Vaser',
423+
order: 1,
424+
type: 'text',
425+
isHiddenFromList: false,
426+
},
427+
},
428+
},
429+
},
430+
})
431+
},
432+
)
433+
434+
it.each(['0', '01', '1', '2', '123', '999', 'a'])(
435+
'should not hide column when the column name happens to be a number: %s',
436+
async (columnName) => {
437+
const testHexColumnId = Buffer.from(columnName).toString('hex')
438+
const dataOut = {
439+
iterations: 1,
440+
inputSource: FOR_EACH_INPUT_SOURCE.M365_EXCEL,
441+
items: {
442+
columns: {
443+
[testHexColumnId]: {
444+
id: testHexColumnId,
445+
name: columnName,
446+
value: `items.rows.__ITERATION__.data.${testHexColumnId}`,
447+
order: 1,
448+
},
449+
},
450+
rows: [{ data: { [testHexColumnId]: 'Vaser' } }],
451+
inputSource: FOR_EACH_INPUT_SOURCE.M365_EXCEL,
452+
},
453+
}
454+
const executionStep = createMockExecutionStep(dataOut)
455+
const result = await getDataOutMetadata(executionStep)
456+
457+
expect(result).toEqual({
458+
...GENERIC_TABLE_EXPECTED_RESULT,
459+
items: {
460+
...GENERIC_TABLE_EXPECTED_RESULT.items,
461+
columns: {
462+
[testHexColumnId]: {
463+
id: { isHidden: true },
464+
name: { isHidden: true },
465+
order: { isHidden: true },
466+
value: {
467+
label: columnName,
468+
displayedValue: 'Vaser',
469+
order: 1,
470+
type: 'text',
471+
isHiddenFromList: false,
472+
},
473+
},
474+
},
475+
},
476+
})
477+
},
478+
)
354479
})
355480

356481
describe('when inputSource is TILES', () => {
@@ -386,16 +511,7 @@ describe('getDataOutMetadata', () => {
386511
const result = await getDataOutMetadata(executionStep)
387512

388513
expect(result).toEqual({
389-
iteration: {
390-
label: 'Item number',
391-
displayedValue: '1',
392-
},
393-
iterations: {
394-
label: 'Items found',
395-
},
396-
inputSource: {
397-
isHidden: true,
398-
},
514+
...GENERIC_TABLE_EXPECTED_RESULT,
399515
items: {
400516
columns: [
401517
{
@@ -460,16 +576,11 @@ describe('getDataOutMetadata', () => {
460576
const result = await getDataOutMetadata(executionStep)
461577

462578
expect(result).toEqual({
579+
...GENERIC_TABLE_EXPECTED_RESULT,
463580
iteration: {
464581
label: 'Item number',
465582
displayedValue: '',
466583
},
467-
iterations: {
468-
label: 'Items found',
469-
},
470-
inputSource: {
471-
isHidden: true,
472-
},
473584
items: {
474585
columns: [
475586
{
@@ -525,16 +636,7 @@ describe('getDataOutMetadata', () => {
525636
const result = await getDataOutMetadata(executionStep)
526637

527638
expect(result).toEqual({
528-
iteration: {
529-
label: 'Item number',
530-
displayedValue: '1',
531-
},
532-
iterations: {
533-
label: 'Items found',
534-
},
535-
inputSource: {
536-
isHidden: true,
537-
},
639+
...GENERIC_TABLE_EXPECTED_RESULT,
538640
items: {
539641
columns: [
540642
{
@@ -605,16 +707,7 @@ describe('getDataOutMetadata', () => {
605707
const result = await getDataOutMetadata(executionStep)
606708

607709
expect(result).toEqual({
608-
iteration: {
609-
label: 'Item number',
610-
displayedValue: '1',
611-
},
612-
iterations: {
613-
label: 'Items found',
614-
},
615-
inputSource: {
616-
isHidden: true,
617-
},
710+
...GENERIC_TABLE_EXPECTED_RESULT,
618711
items: {
619712
columns: [
620713
{
@@ -786,16 +879,7 @@ describe('getDataOutMetadata', () => {
786879
const result = await getDataOutMetadata(executionStep)
787880

788881
expect(result).toEqual({
789-
iteration: {
790-
label: 'Item number',
791-
displayedValue: '1',
792-
},
793-
iterations: {
794-
label: 'Items found',
795-
},
796-
inputSource: {
797-
isHidden: true,
798-
},
882+
...GENERIC_TABLE_EXPECTED_RESULT,
799883
items: {
800884
columns: [
801885
{

‎packages/backend/src/apps/toolbox/actions/for-each/get-data-out-metadata.ts‎

Lines changed: 26 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,19 @@ import {
77

88
import { dataOutSchema } from './schema'
99

10+
// NOTE: this is for backward compatibility with the old dataOut format
11+
const isBackwardCompatibilityColumnId = (id: string, numColumns: number) => {
12+
// must be numeric
13+
// cannot have leading 0
14+
// must only contain digits
15+
// must be less than the number of columns
16+
if (Number(id) < numColumns && /^(0|[1-9]\d*)$/.test(id)) {
17+
return true
18+
}
19+
20+
return false
21+
}
22+
1023
async function getDataOutMetadata(
1124
executionStep: IExecutionStep,
1225
): Promise<IDataOutMetadata> {
@@ -75,22 +88,32 @@ async function getDataOutMetadata(
7588
}))
7689
} else {
7790
const tempColumnsMetadata = {} as Record<string, IDataOutMetadata>
91+
7892
Object.entries(items.columns)
7993
.sort((a, b) => a[1].order - b[1].order)
8094
.forEach(([id, column], index) => {
81-
const isBackwardCompatibilityColumnId = !isNaN(Number(id))
95+
const isBackwardCompat = isBackwardCompatibilityColumnId(
96+
id,
97+
Object.keys(items.columns).length,
98+
)
99+
82100
tempColumnsMetadata[id] = {
83101
id: { isHidden: true },
84102
name: { isHidden: true },
85103
value: {
86-
label: column.name,
104+
label: isBackwardCompat ? `(Fix me) ${column.name}` : column.name,
87105
displayedValue:
88106
column.id === 'rowId'
89107
? items?.rows?.[0]?.rowId ?? ''
90108
: String(items?.rows?.[0]?.data?.[column.id] ?? ''),
91109
order: index + 1,
92110
type: column.id === 'rowId' ? 'tile_row_id' : 'text', // NOTE: only tiles will have rowId
93-
isHiddenFromList: isBackwardCompatibilityColumnId,
111+
/**
112+
* NOTE: this is for backward compatibility with the old dataOut format
113+
*
114+
* TODO (kevinkim-ogp): remove this once all users have moved to the new format
115+
*/
116+
isHiddenFromList: isBackwardCompat,
94117
},
95118
order: { isHidden: true },
96119
}

0 commit comments

Comments
 (0)