In case the connection is not working, we might want to have a warning on /admin/reports/status page using hook_requirements().

CommentFileSizeAuthor
#2 3101303-2-smtp-hook_requirements.patch10.85 KBalberto56

Comments

alberto56 created an issue. See original summary.

alberto56’s picture

alberto56 commented
Status: Active » Needs review
StatusFileSize
new 3101303-2-smtp-hook_requirements.patch10.85 KB

Here is an idea for a patch.

rivimey’s picture

rivimey
she/her
commented
Status: Needs review » Reviewed & tested by the community

Visual check: looks good. I'll test it out.... it seems fine to me.

rivimey’s picture

rivimey
she/her
commented
Parent issue: » #3093579: Plan for SMTP 7.x-1.8 release
rivimey’s picture

rivimey
she/her
commented
Parent issue: #3093579: Plan for SMTP 7.x-1.8 release »
rivimey’s picture

rivimey
she/her
commented
Parent issue: » #3114671: Plan for SMTP 8.x-1.0 release
rivimey’s picture

rivimey
she/her
commented
Parent issue: #3114671: Plan for SMTP 8.x-1.0 release » #3081980: Plan for full 8.x-1.0 release of SMTP
wundo’s picture

wundo commented

Great contribution @alberto56! Thanks :)

  • wundo committed 42fe122 on 8.x-1.x authored by alberto56 
    Issue #3101303 by alberto56, rivimey, wundo: Add hook_requirements() to...
wundo’s picture

wundo commented
wundo’s picture

wundo commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.