Hi people, how are you?

Congrats on the project. XML Sitemap is very very useful for everyone! Thanks!

I update for external links open in new window to prevent escape from Drupal.

CommentFileSizeAuthor
#2 xmlsitemap-added_external_links-2869015-2.patch2.63 KBrenatog
xmlsitemap.png87.98 KBrenatog
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

RenatoG created an issue. See original summary.

renatog’s picture

renatog
Location Campinas
CreditAttribution: renatog at CI&T commented
Assigned: renatog » Unassigned
Status: Active » Needs review
FileSize
xmlsitemap-added_external_links-2869015-2.patch2.63 KB

Fixed.

Patch with fix it's in attachment.

Good Work.

Regards.

rodrigoac’s picture

rodrigoac CreditAttribution: rodrigoac at CI&T commented
Assigned: Unassigned » rodrigoac
rodrigoac’s picture

rodrigoac CreditAttribution: rodrigoac at CI&T commented
Assigned: rodrigoac » Unassigned
Status: Needs review » Reviewed & tested by the community

Hello People !

I check the patch and works good for me !

Thanks @RenatoG !

  • RenatoG committed e2f0595 on 8.x-1.x 
    Issue #2869015 by RenatoG, rodrigoac: Added external links for open in...
renatog’s picture

renatog
Location Campinas
CreditAttribution: renatog at CI&T commented
Status: Reviewed & tested by the community » Fixed

Fixed.

Committed to the dev branch.

Thank you very much.

Regards.

Status: Fixed » Closed (fixed)

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