anthonychengit opened a new pull request, #7639: URL: https://github.com/apache/texera/pull/7639
### What changes were proposed in this PR? The account-request notification only forwarded the requester's email address to the template. Although the stored user record already contains the requester's name, the notification path never retrieved it. ```text Before: account request -> email only -> admin notification omits name After: account request -> stored user lookup -> admin receives name and email ``` This change looks up the stored requester by email and includes the name in the admin notification. Missing or blank names render as `Not provided`. The requester-facing acknowledgement remains unchanged and does not expose the name field. Tests cover the positive stored-name path, the missing-requester fallback, blank and absent names, and the negative requester-facing template case. ### Any related issues, documentation, discussions? Closes #7595 ### How was this PR tested? ```bash sbt "WorkflowExecutionService / Test / testOnly org.apache.texera.web.resource.EmailTemplateSpec org.apache.texera.web.resource.GmailResourceSpec" sbt scalafmtCheckAll sbt "scalafixAll --check" ``` The focused suite passed 12 tests. Scala formatting and Scalafix checks also passed. ### Was this PR authored or co-authored using generative AI tooling? Generated-by: OpenAI Codex (GPT-5) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
