bytemakers / Noteslify

Your Privacy Friendly, Open Source. Alternative to EverNote.
GNU General Public License v3.0
125 stars 78 forks source link

add 500 error email middleware #273

Closed thevinitgupta closed 1 year ago

thevinitgupta commented 1 year ago

Describe your changes

Added a middleware to send email to send email to admin in case of 500 server error

Screenshots - If Any (Optional)

image

Issue ticket number and link - If Any

215

Checklist before requesting a review

Additional Information (Optional)

The following details for the nodemailer configuration needs to be updated with admin details : error js-2023-02-21-13-02-04 error js-2023-02-21-13-02-34

Gauravdarkslayer commented 1 year ago

@devarshishimpi , should we place mail functionality to a separate file(helper.js) to avoid code duplication ? If yes, should this task be covered in this issue or by creating a separate issue?

devarshishimpi commented 1 year ago

@devarshishimpi , should we place mail functionality to a separate file(helper.js) to avoid code duplication ? If yes, should this task be covered in this issue or by creating a separate issue?

I think the same file works

Gauravdarkslayer commented 1 year ago

Alright then! Everything else is looks good for me .