Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added mailto URLs (when appropriate) where email addresses are displayed #4316

Merged
merged 1 commit into from
Jan 3, 2025

Conversation

wes-otf
Copy link
Contributor

@wes-otf wes-otf commented Jan 2, 2025

Fixes #4312. Small fixes to add mailto links in submission detail & wagtail admin user list. I think these are the only places it would make sense to have these URLs to but let me know if I missed anything!

Test Steps

  • Ensure the email address shown under the submission detail is now a functional mailto link
  • Ensure the email column in the wagtail admin user list (/admin/users/) contains all functional mailto links

@wes-otf wes-otf added the Type: Enhancement This is an improvement of an existing thing (not a new thing, which would be a feature). label Jan 2, 2025
@wes-otf wes-otf requested a review from frjo January 2, 2025 21:57
@wes-otf wes-otf added the Type: Patch Mini change, used in release drafter label Jan 2, 2025
@frjo frjo added Status: Needs testing Tickets that need testing/qa Status: Needs dev testing 🧑‍💻 Tasks that should be tested by the dev team Status: Tested - approved for live ✅ and removed Status: Needs testing Tickets that need testing/qa Status: Needs dev testing 🧑‍💻 Tasks that should be tested by the dev team labels Jan 3, 2025
@frjo frjo merged commit 101a107 into main Jan 3, 2025
7 checks passed
@frjo frjo deleted the enhancement/make-all-emails-clickable branch March 11, 2025 19:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Tested - approved for live ✅ Type: Enhancement This is an improvement of an existing thing (not a new thing, which would be a feature). Type: Patch Mini change, used in release drafter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make all e-mails into clickable links
2 participants