td-amp adding amp link for attachment page in attachment page [fix here]

Posted in: Newspaper
Post count: 68
kuaza
#252960
Sep 11, 2018 at 3:48 pm

hi, i found one error in attachment page.

attache

is not good, because in webmaster tools 8000 error amp link :/

for fix:

open: wp-content/plugins/td-amp/includes/amp-frontend-actions.php

find:
add_action( 'wp_head', 'amp_frontend_add_canonical' );

add before:
if( is_singular( array('post','page') ) )

finish, after clean all cache.

Post count: 20688
Simion C.
#253042
Sep 12, 2018 at 9:00 am

Hi,

So the AMP link appears in an attachment page source? That shouldn’t happen. I will pass this along to our developer team, they will take a look and make modification if needed in the plugin.

Thanks

Post count: 68
kuaza
#253161
Sep 12, 2018 at 5:47 pm

yes and for new update: https://forum.tagdiv.com/topic/pinterest-share-icon-change-pinit-js-problem/

thanks, good day

Post count: 68
kuaza
#268471
Dec 20, 2018 at 9:59 pm

this problem came back again. and overturned on top.

tagdiv AMP: 1.3 | built on 12.12.2018 11:28
Themes: Theme version 9.2.2

Post count: 20688
Simion C.
#268540
Dec 21, 2018 at 1:44 pm

Hi,

Quite unusual, there is a condition in the AMP plugin preventing this from happening
– https://www.screencast.com/t/kdNqCoxN9ln
Only by removing this condition would this issue occur. So there would be no relation between attachment pages and AMP, the amphtml link should never appear in attachment pages. I made tests on my end and this is not happening.

Thanks

Post count: 68
kuaza
#268841
Dec 24, 2018 at 11:54 pm

i change this line after work. i added one video : https://youtu.be/o32KDgi8NfA

if ( ! ( is_singular( array('post','page')) || $wp_query->is_posts_page ) || is_feed() ) {

Viewing 6 posts - 1 through 6 (of 6 total)
The forum ‘Newspaper’ is closed to new topics and replies.