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

Implement ability to email admins/users #25

Closed
neilong31 opened this issue Nov 2, 2019 · 3 comments
Closed

Implement ability to email admins/users #25

neilong31 opened this issue Nov 2, 2019 · 3 comments
Projects

Comments

@neilong31
Copy link
Collaborator

@neilong31 neilong31 commented Nov 2, 2019

This issue is a part of the MVP Features.

It should send an email to an admin or the user if:

  • A post is deemed inappropriate
  • Any major errors are logged (ex. issues downloading feeds)
  • If it crashed

If there any other times an email should be sent out please suggest them as well.

@Reza-Rajabi Reza-Rajabi added this to Issue in Main Nov 6, 2019
@Reza-Rajabi Reza-Rajabi moved this from Issue to To do in Main Nov 6, 2019
@s-arika

This comment has been minimized.

Copy link
Collaborator

@s-arika s-arika commented Nov 6, 2019

Maybe we could use nodemailer https://nodemailer.com/about/

@humphd

This comment has been minimized.

Copy link
Contributor

@humphd humphd commented Nov 7, 2019

Sounds good to me, let's try it. https://stackabuse.com/how-to-send-emails-with-node-js/ has a good discussion of this, including how to test it. Who wants to do this?

@RyanWils

This comment has been minimized.

Copy link
Contributor

@RyanWils RyanWils commented Nov 7, 2019

Hi I would like to work on this

RyanWils pushed a commit to RyanWils/telescope that referenced this issue Nov 7, 2019
RyanWils pushed a commit to RyanWils/telescope that referenced this issue Nov 7, 2019
@Reza-Rajabi Reza-Rajabi added the on-board label Nov 8, 2019
RyanWils pushed a commit to RyanWils/telescope that referenced this issue Nov 12, 2019
RyanWils pushed a commit to RyanWils/telescope that referenced this issue Nov 12, 2019
@Reza-Rajabi Reza-Rajabi moved this from To do to In progress/Review in Main Nov 13, 2019
RyanWils pushed a commit to RyanWils/telescope that referenced this issue Nov 14, 2019
RyanWils pushed a commit to RyanWils/telescope that referenced this issue Nov 14, 2019
@RyanWils RyanWils closed this in 261e744 Nov 14, 2019
RyanWils added a commit that referenced this issue Nov 14, 2019
Fix #25: Implement basic email sender
Main automation moved this from In progress/Review to Closed Nov 14, 2019
dbeigi added a commit to dbeigi/telescope that referenced this issue Nov 15, 2019
humphd added a commit that referenced this issue Nov 15, 2019
* fix 115 added test cases

* removed space at the end of file

* addressing check failures

* Fix #25: Implement basic email sender

* Async Function Promise and Env File

* Travis CI Fix

* Travis CI

* Travis CI Fix

* Comments

* Implement SMTP

* Circle Ci Changes

* package.json conflict

* info.accepted

* rework on test cases
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Main
Closed
5 participants
You can’t perform that action at this time.