-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Bounce non-user email replies #9126
Conversation
Codecov Report
@@ Coverage Diff @@
## main #9126 +/- ##
=======================================
Coverage ? 49.68%
=======================================
Files ? 98
Lines ? 6107
Branches ? 0
=======================================
Hits ? 3034
Misses ? 3073
Partials ? 0 |
@jywarren I've used regex to get the email ID and sending an email using |
@jywarren regarding sending back an email in case of other errors, I checked the |
Hi, unfortunately the |
Code Climate has analyzed commit 6f5eb32 and detected 0 issues on this pull request. View more on Code Climate. |
Awesome work, thank you!!! |
* added reply email * resolve merge conflict * trigger build * Update app/models/comment.rb * Update app/models/comment.rb Co-authored-by: Jeffrey Warren <[email protected]>
* added reply email * resolve merge conflict * trigger build * Update app/models/comment.rb * Update app/models/comment.rb Co-authored-by: Jeffrey Warren <[email protected]>
Fixes #3322 (<=== Add issue number here)
Make sure these boxes are checked before your pull request (PR) is ready to be reviewed and merged. Thanks!
rake test
@publiclab/reviewers
for help, in a comment belowIf tests do fail, click on the red
X
to learn why by reading the logs.Please be sure you've reviewed our contribution guidelines at https://publiclab.org/contributing-to-public-lab-software
Thanks!