-
Notifications
You must be signed in to change notification settings - Fork 733
Automations feature backend #43
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
Conversation
✅ Deploy Preview for open-crowd-prod ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
✅ Deploy Preview for open-devfounders ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
fb479e6
to
eaf7f70
Compare
1ca62da
to
c06770c
Compare
c06770c
to
55b836f
Compare
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 for addressing/revisiting so many things already! 🙏
Left another comment for us to think about
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.
Looking good 🚀 I've included some comments and nitpicks
backend/src/database/repositories/automationExecutionRepository.ts
Outdated
Show resolved
Hide resolved
…ction is commited.
Changes proposed ✍️
webhook
type triggered by new activities and new community membersChecklist ✅
type:feature 🚀
,type:enhancement ✨
,type:bug 🐞
, ortype:documentation 📜
.Environment variables have been updatedFront-end:frontend/.env.dist
Backend:backend/.env.dist
,backend/.env.dist.staging
,backend/.env.dist.staging
.Configuration docs have been updated.Team members only: update environment variables in Password manager and update the team