mirror of
https://github.com/n8n-io/n8n.git
synced 2025-03-05 20:50:17 -08:00
fix(editor): Hide not supported node options (#7597)
Currently Trigger-Nodes and Sub-Nodes display options that do not make sense or work like: - Always Output Data - Execute Once - On Error This PR hides those options for nodes that can not make use of them anyway to not confuse users. Github issue / Community forum post (link here to close automatically):
This commit is contained in:
Jan Oberhauser
GitHub
parent
85e1a10021
commit
b532a7bdb7
|
|
@ -380,120 +380,7 @@ export default defineComponent({
|
||||||
} as INodeParameters,
|
} as INodeParameters,
|
||||||
nodeValuesInitialized: false, // Used to prevent nodeValues from being overwritten by defaults on reopening ndv
|
nodeValuesInitialized: false, // Used to prevent nodeValues from being overwritten by defaults on reopening ndv
|
||||||
|
|
||||||
nodeSettings: [
|
nodeSettings: [] as INodeProperties[],
|
||||||
{
|
|
||||||
displayName: this.$locale.baseText('nodeSettings.alwaysOutputData.displayName'),
|
|
||||||
name: 'alwaysOutputData',
|
|
||||||
type: 'boolean',
|
|
||||||
default: false,
|
|
||||||
noDataExpression: true,
|
|
||||||
description: this.$locale.baseText('nodeSettings.alwaysOutputData.description'),
|
|
||||||
},
|
|
||||||
{
|
|
||||||
displayName: this.$locale.baseText('nodeSettings.executeOnce.displayName'),
|
|
||||||
name: 'executeOnce',
|
|
||||||
type: 'boolean',
|
|
||||||
default: false,
|
|
||||||
noDataExpression: true,
|
|
||||||
description: this.$locale.baseText('nodeSettings.executeOnce.description'),
|
|
||||||
},
|
|
||||||
{
|
|
||||||
displayName: this.$locale.baseText('nodeSettings.retryOnFail.displayName'),
|
|
||||||
name: 'retryOnFail',
|
|
||||||
type: 'boolean',
|
|
||||||
default: false,
|
|
||||||
noDataExpression: true,
|
|
||||||
description: this.$locale.baseText('nodeSettings.retryOnFail.description'),
|
|
||||||
},
|
|
||||||
{
|
|
||||||
displayName: this.$locale.baseText('nodeSettings.maxTries.displayName'),
|
|
||||||
name: 'maxTries',
|
|
||||||
type: 'number',
|
|
||||||
typeOptions: {
|
|
||||||
minValue: 2,
|
|
||||||
maxValue: 5,
|
|
||||||
},
|
|
||||||
default: 3,
|
|
||||||
displayOptions: {
|
|
||||||
show: {
|
|
||||||
retryOnFail: [true],
|
|
||||||
},
|
|
||||||
},
|
|
||||||
noDataExpression: true,
|
|
||||||
description: this.$locale.baseText('nodeSettings.maxTries.description'),
|
|
||||||
},
|
|
||||||
{
|
|
||||||
displayName: this.$locale.baseText('nodeSettings.waitBetweenTries.displayName'),
|
|
||||||
name: 'waitBetweenTries',
|
|
||||||
type: 'number',
|
|
||||||
typeOptions: {
|
|
||||||
minValue: 0,
|
|
||||||
maxValue: 5000,
|
|
||||||
},
|
|
||||||
default: 1000,
|
|
||||||
displayOptions: {
|
|
||||||
show: {
|
|
||||||
retryOnFail: [true],
|
|
||||||
},
|
|
||||||
},
|
|
||||||
noDataExpression: true,
|
|
||||||
description: this.$locale.baseText('nodeSettings.waitBetweenTries.description'),
|
|
||||||
},
|
|
||||||
{
|
|
||||||
displayName: this.$locale.baseText('nodeSettings.onError.displayName'),
|
|
||||||
name: 'onError',
|
|
||||||
type: 'options',
|
|
||||||
options: [
|
|
||||||
{
|
|
||||||
name: this.$locale.baseText('nodeSettings.onError.options.stopWorkflow.displayName'),
|
|
||||||
value: 'stopWorkflow',
|
|
||||||
description: this.$locale.baseText(
|
|
||||||
'nodeSettings.onError.options.stopWorkflow.description',
|
|
||||||
),
|
|
||||||
},
|
|
||||||
{
|
|
||||||
name: this.$locale.baseText(
|
|
||||||
'nodeSettings.onError.options.continueRegularOutput.displayName',
|
|
||||||
),
|
|
||||||
value: 'continueRegularOutput',
|
|
||||||
description: this.$locale.baseText(
|
|
||||||
'nodeSettings.onError.options.continueRegularOutput.description',
|
|
||||||
),
|
|
||||||
},
|
|
||||||
{
|
|
||||||
name: this.$locale.baseText(
|
|
||||||
'nodeSettings.onError.options.continueErrorOutput.displayName',
|
|
||||||
),
|
|
||||||
value: 'continueErrorOutput',
|
|
||||||
description: this.$locale.baseText(
|
|
||||||
'nodeSettings.onError.options.continueErrorOutput.description',
|
|
||||||
),
|
|
||||||
},
|
|
||||||
],
|
|
||||||
default: 'stopWorkflow',
|
|
||||||
noDataExpression: true,
|
|
||||||
description: this.$locale.baseText('nodeSettings.onError.description'),
|
|
||||||
},
|
|
||||||
{
|
|
||||||
displayName: this.$locale.baseText('nodeSettings.notes.displayName'),
|
|
||||||
name: 'notes',
|
|
||||||
type: 'string',
|
|
||||||
typeOptions: {
|
|
||||||
rows: 5,
|
|
||||||
},
|
|
||||||
default: '',
|
|
||||||
noDataExpression: true,
|
|
||||||
description: this.$locale.baseText('nodeSettings.notes.description'),
|
|
||||||
},
|
|
||||||
{
|
|
||||||
displayName: this.$locale.baseText('nodeSettings.notesInFlow.displayName'),
|
|
||||||
name: 'notesInFlow',
|
|
||||||
type: 'boolean',
|
|
||||||
default: false,
|
|
||||||
noDataExpression: true,
|
|
||||||
description: this.$locale.baseText('nodeSettings.notesInFlow.description'),
|
|
||||||
},
|
|
||||||
] as INodeProperties[],
|
|
||||||
COMMUNITY_NODES_INSTALLATION_DOCS_URL,
|
COMMUNITY_NODES_INSTALLATION_DOCS_URL,
|
||||||
CUSTOM_NODES_DOCS_URL,
|
CUSTOM_NODES_DOCS_URL,
|
||||||
MAIN_NODE_PANEL_WIDTH,
|
MAIN_NODE_PANEL_WIDTH,
|
||||||
|
|
@ -538,6 +425,132 @@ export default defineComponent({
|
||||||
|
|
||||||
this.workflowsStore.setNodePristine(this.node.name, false);
|
this.workflowsStore.setNodePristine(this.node.name, false);
|
||||||
},
|
},
|
||||||
|
populateSettings() {
|
||||||
|
if (this.isExecutable && !this.isTriggerNode) {
|
||||||
|
this.nodeSettings.push(
|
||||||
|
...([
|
||||||
|
{
|
||||||
|
displayName: this.$locale.baseText('nodeSettings.alwaysOutputData.displayName'),
|
||||||
|
name: 'alwaysOutputData',
|
||||||
|
type: 'boolean',
|
||||||
|
default: false,
|
||||||
|
noDataExpression: true,
|
||||||
|
description: this.$locale.baseText('nodeSettings.alwaysOutputData.description'),
|
||||||
|
},
|
||||||
|
{
|
||||||
|
displayName: this.$locale.baseText('nodeSettings.executeOnce.displayName'),
|
||||||
|
name: 'executeOnce',
|
||||||
|
type: 'boolean',
|
||||||
|
default: false,
|
||||||
|
noDataExpression: true,
|
||||||
|
description: this.$locale.baseText('nodeSettings.executeOnce.description'),
|
||||||
|
},
|
||||||
|
{
|
||||||
|
displayName: this.$locale.baseText('nodeSettings.retryOnFail.displayName'),
|
||||||
|
name: 'retryOnFail',
|
||||||
|
type: 'boolean',
|
||||||
|
default: false,
|
||||||
|
noDataExpression: true,
|
||||||
|
description: this.$locale.baseText('nodeSettings.retryOnFail.description'),
|
||||||
|
},
|
||||||
|
{
|
||||||
|
displayName: this.$locale.baseText('nodeSettings.maxTries.displayName'),
|
||||||
|
name: 'maxTries',
|
||||||
|
type: 'number',
|
||||||
|
typeOptions: {
|
||||||
|
minValue: 2,
|
||||||
|
maxValue: 5,
|
||||||
|
},
|
||||||
|
default: 3,
|
||||||
|
displayOptions: {
|
||||||
|
show: {
|
||||||
|
retryOnFail: [true],
|
||||||
|
},
|
||||||
|
},
|
||||||
|
noDataExpression: true,
|
||||||
|
description: this.$locale.baseText('nodeSettings.maxTries.description'),
|
||||||
|
},
|
||||||
|
{
|
||||||
|
displayName: this.$locale.baseText('nodeSettings.waitBetweenTries.displayName'),
|
||||||
|
name: 'waitBetweenTries',
|
||||||
|
type: 'number',
|
||||||
|
typeOptions: {
|
||||||
|
minValue: 0,
|
||||||
|
maxValue: 5000,
|
||||||
|
},
|
||||||
|
default: 1000,
|
||||||
|
displayOptions: {
|
||||||
|
show: {
|
||||||
|
retryOnFail: [true],
|
||||||
|
},
|
||||||
|
},
|
||||||
|
noDataExpression: true,
|
||||||
|
description: this.$locale.baseText('nodeSettings.waitBetweenTries.description'),
|
||||||
|
},
|
||||||
|
{
|
||||||
|
displayName: this.$locale.baseText('nodeSettings.onError.displayName'),
|
||||||
|
name: 'onError',
|
||||||
|
type: 'options',
|
||||||
|
options: [
|
||||||
|
{
|
||||||
|
name: this.$locale.baseText(
|
||||||
|
'nodeSettings.onError.options.stopWorkflow.displayName',
|
||||||
|
),
|
||||||
|
value: 'stopWorkflow',
|
||||||
|
description: this.$locale.baseText(
|
||||||
|
'nodeSettings.onError.options.stopWorkflow.description',
|
||||||
|
),
|
||||||
|
},
|
||||||
|
{
|
||||||
|
name: this.$locale.baseText(
|
||||||
|
'nodeSettings.onError.options.continueRegularOutput.displayName',
|
||||||
|
),
|
||||||
|
value: 'continueRegularOutput',
|
||||||
|
description: this.$locale.baseText(
|
||||||
|
'nodeSettings.onError.options.continueRegularOutput.description',
|
||||||
|
),
|
||||||
|
},
|
||||||
|
{
|
||||||
|
name: this.$locale.baseText(
|
||||||
|
'nodeSettings.onError.options.continueErrorOutput.displayName',
|
||||||
|
),
|
||||||
|
value: 'continueErrorOutput',
|
||||||
|
description: this.$locale.baseText(
|
||||||
|
'nodeSettings.onError.options.continueErrorOutput.description',
|
||||||
|
),
|
||||||
|
},
|
||||||
|
],
|
||||||
|
default: 'stopWorkflow',
|
||||||
|
noDataExpression: true,
|
||||||
|
description: this.$locale.baseText('nodeSettings.onError.description'),
|
||||||
|
},
|
||||||
|
] as INodeProperties[]),
|
||||||
|
);
|
||||||
|
}
|
||||||
|
this.nodeSettings.push(
|
||||||
|
...([
|
||||||
|
{
|
||||||
|
displayName: this.$locale.baseText('nodeSettings.notes.displayName'),
|
||||||
|
name: 'notes',
|
||||||
|
type: 'string',
|
||||||
|
typeOptions: {
|
||||||
|
rows: 5,
|
||||||
|
},
|
||||||
|
default: '',
|
||||||
|
noDataExpression: true,
|
||||||
|
description: this.$locale.baseText('nodeSettings.notes.description'),
|
||||||
|
},
|
||||||
|
{
|
||||||
|
displayName: this.$locale.baseText('nodeSettings.notesInFlow.displayName'),
|
||||||
|
name: 'notesInFlow',
|
||||||
|
type: 'boolean',
|
||||||
|
default: false,
|
||||||
|
noDataExpression: true,
|
||||||
|
description: this.$locale.baseText('nodeSettings.notesInFlow.description'),
|
||||||
|
},
|
||||||
|
] as INodeProperties[]),
|
||||||
|
);
|
||||||
|
},
|
||||||
onParameterBlur(parameterName: string) {
|
onParameterBlur(parameterName: string) {
|
||||||
this.hiddenIssuesInputs = this.hiddenIssuesInputs.filter((name) => name !== parameterName);
|
this.hiddenIssuesInputs = this.hiddenIssuesInputs.filter((name) => name !== parameterName);
|
||||||
},
|
},
|
||||||
|
|
@ -1003,6 +1016,7 @@ export default defineComponent({
|
||||||
},
|
},
|
||||||
mounted() {
|
mounted() {
|
||||||
this.populateHiddenIssuesSet();
|
this.populateHiddenIssuesSet();
|
||||||
|
this.populateSettings();
|
||||||
this.setNodeValues();
|
this.setNodeValues();
|
||||||
this.eventBus?.on('openSettings', this.openSettings);
|
this.eventBus?.on('openSettings', this.openSettings);
|
||||||
|
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue