seriou5

Hi Pedro,

thank you – after contacting Salient Theme Authors and playing around with the Health Troubleshoot Plugin we found out that the Object Cache in was the problem.

Everything good now – thank you.

Best regards

Thanks a lot – it is working now ??

Hi, we have the same problem – please fix alt-attributes for AMP images. It is important for On-Page SEO.

Sorry, update: we saw that the missing alt-tags seem to be related to “ tag without alt. But this should not be a problem.

• This reply was modified 4 years, 2 months ago by seriou5.
• This reply was modified 4 years, 2 months ago by seriou5.

@ahmedkaludi anything new regarding this topic?

sure, here we go: https://ibb.co/R0NMKCX

We link our GTM. In GTM we are loading the GA Code – never changed a thing since we activated the GDPR Popup.

Hi @ampforwp

no, the wp-worthy plugin is adding pixels only to posts.

Is there a similar check for posts too?

Hi @ampforwp,

I will try!

wp-worthy uses the ‘the_content’ hook to filter the content and add a pixel to the output:

add_filter ('the_content', array ($this, 'onContent'));

Inside the ‘onContent’ function another function (‘addMarker’) is triggered only IF the main query is used. It is checked with:

if (!$this->onMainQuery)
     return $content;

When using AMPforWP this condition is not passed (= we are not on the main query) and so the “addMarker” function inside the “onContent” function is never triggered.

As soon as I delete the above mentioned condition the pixel is added successfully.

If you want I can also provide you with the relevant file (or you download wp-worthy on your own and check the onContent function starting on line 466 inside the main plugin file).

Thank you

Thank you!

Nearly there…

The 3rd party plugin (wp-worthy) is adding an amp-pixel tag. It’s working fine for the automattic amp-plugin but unfortunately not yet for the AMPforWP plugin, which we are using.

I found out that the plugin is checking for the main-query and only appends the pixel if the main query is used. It is checking this with “is_main_query()” and I assume that AMPforWP is using a custom or secondary query.

Can you point me in the right direction?

Thank you!

• This reply was modified 5 years, 9 months ago by seriou5.

@ampforwp

you got mail

thx it works since the update ??

Bought the premium membership. Same problem here – one of our very big sites is dealing with this problem right now and we even got top ranked articles cleaned out of the SERPS because the URLs changed and nobody noticed it. Come on guys – don’t mess up core functions… please fix and/or let us know about a workaround asap!

Great, thanks for the fast reply! Worked ??

to clarify the example –> an existing subscriber uses the same e-mail but updates a field like name or a hidden field