From 9da1bfeea14a49b5987fd2a76910ecbcde83a5c6 Mon Sep 17 00:00:00 2001 From: Rehan Ansari Date: Mon, 20 Oct 2025 13:21:52 +0530 Subject: [PATCH] fix: set job owner as reply-to for job emails --- lms/lms/api.py | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/lms/lms/api.py b/lms/lms/api.py index 8071f66c..c5f51daf 100644 --- a/lms/lms/api.py +++ b/lms/lms/api.py @@ -1686,15 +1686,19 @@ def get_job_applications(job): @frappe.whitelist() def send_email_to_applicant(applicant_email, subject, message, job): """Send email to job applicant. Only job owners or system managers can send emails.""" + job_owner = frappe.db.get_value("Job Opportunity", job, "owner") + if "System Manager" not in frappe.get_roles(): - job_owner = frappe.db.get_value("Job Opportunity", job, "owner") if job_owner != frappe.session.user: frappe.throw(_("You don't have permission to send emails for this job")) + job_owner_email = frappe.db.get_value("User", job_owner, "email") + frappe.sendmail( recipients=[applicant_email], subject=subject, message=message, + reply_to=job_owner_email, now=True, )