Skip to content
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

Add E-goi Node #1224

Closed
wants to merge 1 commit into from
Closed

Add E-goi Node #1224

wants to merge 1 commit into from

Conversation

fbobiano
Copy link
Contributor

@fbobiano fbobiano commented Dec 3, 2020

Actions of E-goi node:

  • Create new contact to an existing list of contacts.
  • Get all contacts from an existing list of contacts.
  • Get a specific contact from an existing list of contacts.
  • Update a contact / Attach tags.

@CLAassistant
Copy link

CLAassistant commented Dec 3, 2020

CLA assistant check
All committers have signed the CLA.

@RicardoE105
Copy link
Contributor

@fbobiano really cool. I Will review as soon as I have time and provide feedback. Thanks.

RicardoE105 added a commit that referenced this pull request Dec 3, 2020
@RicardoE105
Copy link
Contributor

@fbobiano just reviewed, and it seems to be working as expected. However, I made some changes to make it work similarly to the other nodes. The changes can be found here #1227. Thanks for your contribution.

janober added a commit that referenced this pull request Dec 19, 2020
* Add E-goi Node

* ⚡ Improvements to #1224

* ⚡ Small improvements

* ⚡ Minor improvements to Egoi-Node

* ⚡ Add 'simple' option to get and getAll operations.

* ⚡ Minor improvements to Egoi-Node

Co-authored-by: fbobiano <fbobiano@e-goi.com>
Co-authored-by: Jan Oberhauser <jan.oberhauser@gmail.com>
@janober
Copy link
Member

janober commented Jan 7, 2021

Sorry. Forgot to update here. Got merged with #1227 and released with n8n@0.99.0

Thanks a lot for your contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants