fix: include website/company name in compliance-check email subject
This commit is contained in:
@@ -325,11 +325,16 @@ async def _run_compliance_check(check_id: str, req: ComplianceCheckRequest):
|
|||||||
profile_html = _build_profile_html(profile)
|
profile_html = _build_profile_html(profile)
|
||||||
full_html = summary_html + profile_html + report_html
|
full_html = summary_html + profile_html + report_html
|
||||||
|
|
||||||
# Step 6: Send email
|
# Step 6: Send email — include website/company name in subject
|
||||||
doc_count = len([r for r in results if not r.error])
|
doc_count = len([r for r in results if not r.error])
|
||||||
|
site_name = (
|
||||||
|
extracted_profile.get("company_profile", {}).get("companyName")
|
||||||
|
or _extract_domain(doc_entries)
|
||||||
|
or "Unbekannt"
|
||||||
|
)
|
||||||
email_result = send_email(
|
email_result = send_email(
|
||||||
recipient=req.recipient,
|
recipient=req.recipient,
|
||||||
subject=f"[COMPLIANCE-CHECK] {doc_count} Dokumente geprueft",
|
subject=f"[COMPLIANCE-CHECK] {site_name} — {doc_count} Dokumente geprueft",
|
||||||
body_html=full_html,
|
body_html=full_html,
|
||||||
)
|
)
|
||||||
|
|
||||||
@@ -457,6 +462,17 @@ async def _check_single(
|
|||||||
)
|
)
|
||||||
|
|
||||||
|
|
||||||
|
def _extract_domain(doc_entries: list[dict]) -> str | None:
|
||||||
|
"""Extract domain name from first URL for email subject."""
|
||||||
|
for entry in doc_entries:
|
||||||
|
url = entry.get("url", "")
|
||||||
|
if url and "://" in url:
|
||||||
|
from urllib.parse import urlparse
|
||||||
|
host = urlparse(url).netloc
|
||||||
|
return host.replace("www.", "") if host else None
|
||||||
|
return None
|
||||||
|
|
||||||
|
|
||||||
def _get_skip_types(profile) -> dict[str, str]:
|
def _get_skip_types(profile) -> dict[str, str]:
|
||||||
"""Doc_types to skip entirely. Currently empty — we check everything
|
"""Doc_types to skip entirely. Currently empty — we check everything
|
||||||
and flag irrelevant items as INFO instead of skipping."""
|
and flag irrelevant items as INFO instead of skipping."""
|
||||||
|
|||||||
Reference in New Issue
Block a user