rjgoyln opened a new pull request, #73271: URL: https://github.com/apache/airflow/pull/73271
## Summary `mark_success_url` has returned `log_url` since #48996 removed the Airflow 2 `/confirm` deep link, but the labels around it stayed. A failure email offers a "Mark success" link next to a "Log" link that opens the same task instance page, so the action reads as broken rather than duplicated. ## Change - Drop the `Mark success:` line from the default alert email body and the docs example. - Hide the SMTP notifier's `Mark Success Link:` row when it would repeat the log link. `mark_success_url` stays a working alias so custom templates that reference it keep rendering. The SMTP template also serves Airflow 2.11, where the property still returns a real `/confirm?...&state=success` URL, so the row is hidden only when the two URLs match rather than dropped outright. related: #73221, which redesigns the same templates --- ##### Was generative AI tooling used to co-author this PR? - [X] Yes — Claude Code (Opus 5) Generated-by: Claude Code (Opus 5) following [the guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions) -- 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]
