-
Notifications
You must be signed in to change notification settings - Fork 12
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
$400 Bounty: Add Field IDs to n8n Airtable Node #19
Comments
close #19 |
/boss $400 verify |
Hi @pnogemane Yep happy for you to attempt this. We'll pay the bounty once this is merged into the n8n main branch and works :) |
Noted I have reopened the pull request. I am waiting for the reviews to be completed and I will advise as soon as the branch is tested and merged |
The branch has been scheduled for review. I will keep you updated on the progress |
Hi @anglilian anglilian I have implemented the required changes as per pull request and a review has been conducted. The branch is now able merge. |
Great thanks for the update that it is ready to merge! We'll pay the bounty once it is actually merged, as it will require approval by the n8n team etc. :) |
Hello @domdomegg the branch has been merged successfully - |
/boss $400 |
Merged See pull Request : pnogemane/n8n-Add-option-to-use-field-IDs-in-Airtable-node#2 |
Hi @pnogemane, we'll pay the bounty once this is merged into the n8n main branch and works :) I believe you've merged it into your own copy of the repo, not the n8n branch. Merging it into n8n will likely require the approval of the n8n maintainers. |
Hello @domdomegg I hope you are well. This is the final stage of the work. I sent through a code review and it was reviewed and approved by the maintainers. The changes had to then wait in line before being merged to the n8n branch. But finally they where. At this point I think its fair to pay. Thank you and much appriciated. |
Hi @pnogemane, as far as I can see you appear to have merged a branch into your own repo, but the one into the n8n branch has not been merged. I might have misunderstood what has happened - could you clarify where the code has been merged into the n8n branch itself? |
I’d like to request an enhancement for the Airtable node in n8n. Currently, the node requires field names when creating or updating records, but Airtable’s API uses field IDs. This can cause issues, especially when field names change or contain special characters, breaking workflows.
A similar functionality for targeting tables by ID instead of table name has been discussed and implemented (see this thread), and having a similar feature for fields would be extremely useful. There was a similar feature request here 1, but it hasn’t received much traction.
Proposed Solution:
Functional Requirements
Compatibility & Documentation
Testing
Review
Bounty: I am offering $400 for this, but I am open to negotiation based on complexity and feedback. If you are interested or would like clarification, please reach out [email protected]
The text was updated successfully, but these errors were encountered: