"Configure" link of the module is taking user to the second tab (Settings) "/admin/config/people/login-destination/settings".
It should ideally take to the first tab "Login destinations" "/admin/config/people/login-destination".

Made changes for the same, needs to be reviewed.

Thanks.

CommentFileSizeAuthor
#2 Selection_397.png35.5 KBmiroslav-lee
configure-link-url.patch394 bytesprashant.c

Comments

Prashant.c created an issue. See original summary.

miroslav-lee’s picture

miroslav-lee
Location Krakow
commented
StatusFileSize
new Selection_397.png35.5 KB

I checked it works for me. But I think if we removed the link to the settings page from Configure, then maybe we should add it to the help page?
screenshot

rsvelko’s picture

rsvelko commented

Well I agree with this proposal, namely to make the Configure link go to 1st tab List, and not to 2nd tab Settings.

This patch/issue is not about removing the link, but changing its destination.

  • rsvelko committed 0d1a325 on 8.x-1.x authored by Prashant.c 
    Issue #2996358 by Prashant.c: Configure link ideally should take user to...
rsvelko’s picture

rsvelko commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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

chris matthews’s picture

chris matthews commented
Related issues: +#3106146: Plan for next Login Destination 8.x-1.x Release