There is no configure link in the module , it would be helpful if we add the link in the code base

CommentFileSizeAuthor
#7 no_configure_link_in-2876832-7.patch380 bytespankajsachdeva
#5 add more error.PNG102.92 KBpankajsachdeva
#2 2876832-2.patch379 bytesPavan B S
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Pavan B S created an issue. See original summary.

Pavan B S’s picture

Pavan B S CreditAttribution: Pavan B S at Valuebound commented
Assigned: Pavan B S » Unassigned
Status: Active » Needs review
FileSize
2876832-2.patch379 bytes

Applying the patch, please review

pankajsachdeva’s picture

pankajsachdeva CreditAttribution: pankajsachdeva as a volunteer and at ]init[ AG commented

Hi Pavan,

The patch is looking good.

pankajsachdeva’s picture

pankajsachdeva CreditAttribution: pankajsachdeva as a volunteer and at ]init[ AG commented
Status: Needs review » Reviewed & tested by the community
pankajsachdeva’s picture

pankajsachdeva CreditAttribution: pankajsachdeva as a volunteer and at ]init[ AG commented
FileSize
add more error.PNG102.92 KB

Sorry I applied the patch and I am getting error

pankajsachdeva’s picture

pankajsachdeva CreditAttribution: pankajsachdeva as a volunteer and at ]init[ AG commented
Assigned: Unassigned » pankajsachdeva
pankajsachdeva’s picture

pankajsachdeva CreditAttribution: pankajsachdeva as a volunteer and at ]init[ AG commented
FileSize
no_configure_link_in-2876832-7.patch380 bytes

I am adding a new working patch.

pankajsachdeva’s picture

pankajsachdeva CreditAttribution: pankajsachdeva as a volunteer and at ]init[ AG commented
Status: Reviewed & tested by the community » Needs review
pankajsachdeva’s picture

pankajsachdeva CreditAttribution: pankajsachdeva as a volunteer and at ]init[ AG commented
Assigned: pankajsachdeva » Unassigned
dhruveshdtripathi’s picture

dhruveshdtripathi CreditAttribution: dhruveshdtripathi at DevsAdda commented
Status: Needs review » Reviewed & tested by the community

Patch #7 is OK according to coding standards. Changing status to RTBC.

mikejw’s picture

mikejw CreditAttribution: mikejw at University of Adelaide commented
Status: Reviewed & tested by the community » Postponed (maintainer needs more info)

Hi guys, thanks for your work - the link is there though under Configuration > Content authoring > Add more alternate settings.

(/admin/config/content/add-more-alternate-settings)

Are you not seeing this?

dhruveshdtripathi’s picture

dhruveshdtripathi CreditAttribution: dhruveshdtripathi at DevsAdda commented

Hello

You did not understand the issue actually. Yes we are seeing that configuration settings page is there, but that is not the issue. The issue is about configure link on module listing page.

Thank you

mikejw’s picture

mikejw CreditAttribution: mikejw at University of Adelaide commented

Ahhh got you, thanks.

  • pankajsachdeva authored f3e8161 on 8.x-1.x 
    Issue #2876832 by pankajsachdeva, Pavan B S: No configure link in module...
mikejw’s picture

mikejw CreditAttribution: mikejw at University of Adelaide commented
Status: Postponed (maintainer needs more info) » Fixed

Status: Fixed » Closed (fixed)

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