Problem/Motivation

Link to the Dashboard is missing on the Extend page.

Steps to reproduce

Go to /admin/modules and Check the "Mail Safety" module.

Proposed resolution

Add route identifier in .info.yml file in order to appear link on Extend page.

CommentFileSizeAuthor
#5 mail_safety-add-dashboard-link-3167966-3.patch266 bytesjasonluttrell
#3 mail_safety-add-dashboard-link-3167966-3.patch266 bytesjasonluttrell
#2 3167966-2.patch266 bytesshashwat purav

Comments

Shashwat Purav created an issue. See original summary.

shashwat purav’s picture

shashwat purav commented
Assigned: shashwat purav » Unassigned
Status: Needs work » Needs review
StatusFileSize
new 3167966-2.patch266 bytes

Added Dashboard link in .info.yml and patch created.

jasonluttrell’s picture

jasonluttrell
Location Washington, DC area
commented
StatusFileSize
new mail_safety-add-dashboard-link-3167966-3.patch266 bytes

Thanks for sharing. This is just a minor tweak so that it applies cleanly.

jasonluttrell’s picture

jasonluttrell
Location Washington, DC area
commented
jasonluttrell’s picture

jasonluttrell
Location Washington, DC area
commented
StatusFileSize
new mail_safety-add-dashboard-link-3167966-3.patch266 bytes

(Sorry, did not intend to post twice.)

I tested the patch and, other than the issue with applying it, it works fine. Thanks again.

  • Nitebreed committed 859c97f on 8.x-1.x 
    Issue #3167966 by jasonluttrell, Shashwat Purav: Add Dashboard link in ....
nitebreed’s picture

nitebreed
Primary language Dutch
commented
Status: Needs review » Fixed
sjerdo’s picture

sjerdo
Primary language Dutch
Location Haarlem
commented

This has been replaced by the route to the mail safety settings page instead of the mail safety dashboard home.

sjerdo’s picture

sjerdo
Primary language Dutch
Location Haarlem
commented
Related issues: +#3058556: Add configure route in info file

Status: Fixed » Closed (fixed)

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