-
Notifications
You must be signed in to change notification settings - Fork 7.6k
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
feat(n8n Google My Business Node): New node #10504
feat(n8n Google My Business Node): New node #10504
Conversation
Hey @valentina98, Thanks for the PR, We have created "GHC-178" as the internal reference to get this reviewed. One of us will be in touch if there are any changes needed, in most cases this is normally within a couple of weeks but it depends on the current workload of the team. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @ @valentina98 for the PR 🙏
the code is easy to read and very organised 👍
left some small refactoring comments apart from that looks good 🎉
packages/nodes-base/nodes/Google/MyBusiness/GoogleMyBusinessTrigger.node.ts
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🎉
Got released with |
Summary
Adds Google My Business node and trigger node
Related Linear tickets, Github issues, and Community forum posts
https://linear.app/n8n/issue/NODE-1614/community-pr-wip-add-google-my-business-node
https://community.n8n.io/t/google-my-business-node-request/8248/10