-
Notifications
You must be signed in to change notification settings - Fork 9.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Properties that depend on other properties #126
Comments
@Narretz Thanks a lot! I agree that this would be a very important feature. Can you, however, please post it in the forum here: Github issues should really only be created to report bugs and not for questions or feature requests. They should all be collected in the forum to have them all in one spot. If we have them here and there things start to get messy. Thanks a lot! So are closing this issue. |
Oh okay. Then ou should really remove "Feature Request" from the list you see in "New Issue" (https://github.com/n8n-io/n8n/tree/af5c7e75fac8ed23ac01e77baed9ecdf09e93055/.github/ISSUE_TEMPLATE). It doesn't make any sense to list "Feature Request" there just to tell you in the template that you shouldn't open a Feature Request. |
I actually disagree there. I added it on purpose. Because if it would not be there, people would not know where to post feature requests to. Then they would either not add it at all and leave or they would simply select anything else and post it here anyway. By having the "Feature Request" option I have at least the chance to guide the people. They click on it, read what the template says and then post it to the forum. |
But it's sending a confusing message. People often don't read the template texts. Or if they do, they might be confused by the fact that the feature request template is telling them not to open a feature request. I was in fact, and thought this was an error / left over from a previous version. Because why would the option for Feature Request tell me not to open one? |
It looks like you can redirect users to an external site instead of prompting them to open a blank issue: https://help.github.com/en/github/building-a-strong-community/configuring-issue-templates-for-your-repository I think this is the better solution. |
Thats great, thanks a lot! I was not aware of that. Got changed. |
Hi,
sometimes the value(s) of a node property are dependant on the value of another node property. For example in Asana, there exist both "Workspaces" and "Projects". In the existing Asana integration, https://github.com/n8n-io/n8n/blob/af5c7e75fac8ed23ac01e77baed9ecdf09e93055/packages/nodes-base/nodes/Asana/Asana.node.ts, the workspaces are loaded with loadOptions (works well), but there is no property for "project". Now I tried to add it, but it's not that easy, because in order to get the relevant project ids, you need the workspace id.
So there needs to be a way to mark a node property as dependant on another property. If property B is dependant on A, then if A changes, the loadOptions function is called (again). I realize that this is not trivial, as you need to check depth of depdency / circular dependency etc., but it's quite necessary in my opinion.
The text was updated successfully, but these errors were encountered: