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

email notification in case slack is not available in the environment #24

Open
nadimrizk83 opened this issue Feb 27, 2020 · 2 comments
Open
Assignees
Labels
enhancement New feature or request

Comments

@nadimrizk83
Copy link

Currently e-mail notifications works for failures (builds, deployments, preview, etc). Would it be possible to add e-mail notifications for the success criteria when build completes, pull request is open, pull request is merged, deployment in progress, and deployment complete?

The reason I am asking for this is because collaboration tools such as Slack is not available or allowed in the environment I currently work with and we would rather use e-mail notifications for now.

Please advise.

@bmoers bmoers self-assigned this Mar 5, 2020
@bmoers bmoers added the enhancement New feature or request label Mar 5, 2020
@bmoers
Copy link
Owner

bmoers commented Mar 5, 2020

You can override the Slack object in cicdInt.js

Slack: require("./slack"),
to send email instead.
Just make sure it implements the same functions as in slack.js
return {

Would that work for you?

@nadimrizk83
Copy link
Author

Thank you very much for coming back to me on this; I would like to keep the slack functionality - I really like this integration in terms of collaboration and for future use - I would like to actually add the e-mail functionality rather than an override. In addition, can you please show me or provide me with the sample code example for implementing this as unfortunately I don't have the development background and skills required to make these code changes - I am more on the system integration side where I am trying to make this work for the team in place - I made it across various instances and I am very pleased with the results so far; but the only thing I am currently missing is the notification piece. If you can assist with this enhancement I would greatly appreciate it! Thank you again for your time.

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

No branches or pull requests

2 participants