mudhorn/n8n
1
0
Fork 0
mirror of https://github.com/n8n-io/n8n.git synced 2024-11-09 22:24:05 -08:00
Code Issues Actions Packages Projects Releases Wiki Activity

fix(Notion Node): Fix broken Notion node parameters (#7864)

Browse source 
We've introduced a new version(`2.1` of Notion node in #7791 but not all
`diplayOptions` conditions were updated. This would effectively prevent
most of the required Notion fields from loading.
This PR adds the new version to all `@version` display conditions which
contain version `2`.


https://github.com/n8n-io/n8n/assets/12657221/4254c646-43b6-46b3-adcc-1b17746901da


Github issue / Community forum post (link here to close automatically):

---------

Signed-off-by: Oleg Ivaniv <me@olegivaniv.com>
Co-authored-by: Michael Kret <michael.k@radency.com>
This commit is contained in:
oleg 2023-11-29 15:11:54 +01:00 committed by GitHub
parent 6d9342e676
commit 51d1f5b820
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
4 changed files with 51 additions and 17 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
packages/nodes-base/nodes/Notion/DatabaseDescription.ts
View file

@ -8,9 +8,11 @@ export const databaseOperations: INodeProperties[] = [
noDataExpression: true,
displayOptions: {
show: {
'@version': [2],
resource: ['database'],
},
hide: {
'@version': [1],
},
},
options: [
{
@ -178,10 +180,12 @@ export const databaseFields: INodeProperties[] = [
type: 'boolean',
displayOptions: {
show: {
'@version': [2],
resource: ['database'],
operation: ['getAll', 'get'],
},
hide: {
'@version': [1],
},
},
default: true,
description: 'Whether to return a simplified version of the response instead of the raw data',

28
packages/nodes-base/nodes/Notion/DatabasePageDescription.ts
View file

@ -14,9 +14,11 @@ export const databasePageOperations: INodeProperties[] = [
noDataExpression: true,
displayOptions: {
show: {
'@version': [2],
resource: ['databasePage'],
},
hide: {
'@version': [1],
},
},
options: [
{
@ -161,10 +163,12 @@ export const databasePageFields: INodeProperties[] = [
default: '',
displayOptions: {
show: {
'@version': [2],
resource: ['databasePage'],
operation: ['create'],
},
hide: {
'@version': [1],
},
},
description: 'Page title. Appears at the top of the page and can be found via Quick Find.',
},
@ -502,9 +506,11 @@ export const databasePageFields: INodeProperties[] = [
},
displayOptions: {
show: {
'@version': [2],
type: ['files'],
},
hide: {
'@version': [1],
},
},
default: {},
options: [
@ -969,9 +975,11 @@ export const databasePageFields: INodeProperties[] = [
},
displayOptions: {
show: {
'@version': [2],
type: ['files'],
},
hide: {
'@version': [1],
},
},
default: {},
options: [
@ -1097,10 +1105,12 @@ export const databasePageFields: INodeProperties[] = [
],
displayOptions: {
show: {
'@version': [2],
resource: ['databasePage'],
operation: ['get'],
},
hide: {
'@version': [1],
},
},
description: 'The Notion Database Page to get',
},
@ -1110,10 +1120,12 @@ export const databasePageFields: INodeProperties[] = [
type: 'boolean',
displayOptions: {
show: {
'@version': [2],
resource: ['databasePage'],
operation: ['get'],
},
hide: {
'@version': [1],
},
},
default: true,
description: 'Whether to return a simplified version of the response instead of the raw data',
@ -1254,10 +1266,12 @@ export const databasePageFields: INodeProperties[] = [
type: 'boolean',
displayOptions: {
show: {
'@version': [2],
'/resource': ['databasePage'],
'/operation': ['getAll'],
},
hide: {
'@version': [1],
},
},
default: false,
description: "Whether to download a file if a database's field contains it",

20
packages/nodes-base/nodes/Notion/GenericFunctions.ts
View file

@ -936,10 +936,12 @@ export function getSearchFilters(resource: string) {
],
displayOptions: {
show: {
'@version': [2],
resource: [resource],
operation: ['getAll'],
},
hide: {
'@version': [1],
},
},
default: 'none',
},
@ -959,11 +961,13 @@ export function getSearchFilters(resource: string) {
],
displayOptions: {
show: {
'@version': [2],
resource: [resource],
operation: ['getAll'],
filterType: ['manual'],
},
hide: {
'@version': [1],
},
},
default: 'anyFilter',
},
@ -976,11 +980,13 @@ export function getSearchFilters(resource: string) {
},
displayOptions: {
show: {
'@version': [2],
resource: [resource],
operation: ['getAll'],
filterType: ['manual'],
},
hide: {
'@version': [1],
},
},
default: {},
placeholder: 'Add Condition',
@ -999,11 +1005,13 @@ export function getSearchFilters(resource: string) {
type: 'notice',
displayOptions: {
show: {
'@version': [2],
resource: [resource],
operation: ['getAll'],
filterType: ['json'],
},
hide: {
'@version': [1],
},
},
default: '',
},
@ -1013,11 +1021,13 @@ export function getSearchFilters(resource: string) {
type: 'string',
displayOptions: {
show: {
'@version': [2],
resource: [resource],
operation: ['getAll'],
filterType: ['json'],
},
hide: {
'@version': [1],
},
},
default: '',
},

12
packages/nodes-base/nodes/Notion/PageDescription.ts
View file

@ -43,9 +43,11 @@ export const pageOperations: INodeProperties[] = [
noDataExpression: true,
displayOptions: {
show: {
'@version': [2],
resource: ['page'],
},
hide: {
'@version': [1],
},
},
options: [
{
@ -127,10 +129,12 @@ export const pageFields: INodeProperties[] = [
],
displayOptions: {
show: {
'@version': [2],
resource: ['page'],
operation: ['archive'],
},
hide: {
'@version': [1],
},
},
description: 'The Notion Page to archive',
},
@ -140,10 +144,12 @@ export const pageFields: INodeProperties[] = [
type: 'boolean',
displayOptions: {
show: {
'@version': [2],
resource: ['page'],
operation: ['archive'],
},
hide: {
'@version': [1],
},
},
default: true,
description: 'Whether to return a simplified version of the response instead of the raw data',