Skip to content

Commit

Permalink
fix: check Mail Domain instead of Mail Domain Request for verified do…
Browse files Browse the repository at this point in the history
…mains
  • Loading branch information
krantheman committed Feb 6, 2025
1 parent c9bf187 commit c9457e2
Showing 1 changed file with 10 additions and 10 deletions.
20 changes: 10 additions & 10 deletions mail/api/admin.py
Original file line number Diff line number Diff line change
Expand Up @@ -24,17 +24,17 @@ def create_tenant(tenant_name: str) -> None:
def get_domain_request(domain_name: str, mail_tenant: str) -> "MailDomainRequest":
"""Fetches Mail Domain Request for a given domain name if it exists, and creates a new one if not"""

if frappe.db.exists("Mail Domain", domain_name):
frappe.throw(_("Domain {0} has already been registered.").format(domain_name))

if name := frappe.db.exists("Mail Domain Request", {"domain_name": domain_name, "tenant": mail_tenant}):
domain_request = frappe.get_doc("Mail Domain Request", name)
if domain_request.is_verified:
frappe.throw(_("Domain {0} is already verified and added.").format(name))

else:
domain_request = frappe.new_doc("Mail Domain Request")
domain_request.domain_name = domain_name
domain_request.tenant = mail_tenant
domain_request.user = frappe.session.user
domain_request.insert()
return frappe.get_doc("Mail Domain Request", name)

domain_request = frappe.new_doc("Mail Domain Request")
domain_request.domain_name = domain_name
domain_request.tenant = mail_tenant
domain_request.user = frappe.session.user
domain_request.insert()

return domain_request

Expand Down

0 comments on commit c9457e2

Please sign in to comment.