search

inboundnow / inbound-pro

Inbound Pro Plugin for WordPress - Landing Page Framework, Calls to Action Framework, Visitor Analytics & List Segmentation, Email Campaigns & Marketing Automation
https://www.inboundnow.com
31 stars 17 forks source link

Request admin-ajax.php #180

Closed kevin-colleter closed 3 years ago

kevin-colleter commented 6 years ago

Hello,

I have a question about the request : admin-ajax.php As you can see below the TTFB is very bad because of this request : github2

There is a lot of CTA impressions : github

One solution was to comment the content of wp_cta_record_impressions, but do you have a better solution to reduce the TTFB ? Can we avoid this part if we don't have any variations ?

atwellpub commented 6 years ago

I believe you've uncovered a bug. I didn't notice it because it's only active when Calls to Action is run by itself. It's not an issue when Inbound Pro is loading Calls to Action because the tracking system differs a little bit.

I'll just tagged commit that should fix it. I also just pushed the changes to WordPress.org. Will you check and confirm?

atwellpub commented 6 years ago

There's also a setting in wp-admin->CTA->Settings that will kill variation loading and default all CTAs to variation A. But it can't stop using AJAX for recording CTA impressions.

kevin-colleter commented 6 years ago

Hey Atwellpub, The release 3.3.6 will resolve this bug ? Which files have been modified, or can you just send me the commit ID please :-) Thank you !

atwellpub commented 6 years ago

Heya yes it should. Here's the commit: https://github.com/inboundnow/inbound-pro/commit/6a041d406bf1801da11965ca7e9e080e3878c3ba