AMPXF - AMP for Xenforo 2 [Paid] 1.4.9

mazzly · May 5, 2022

mazzly updated AMPXF - AMP for Xenforo 2 with a new update entry:

1.4.7 Patch Level 4

Adds missing index over the tries column to speedup queries when fixing unknown images

SatGuyScott · May 9, 2022

Heads up... wile trying to grab the update from your server.

Whenever I log in I get the following error from your server.

Forbidden (403)

CSRF verification failed. Request aborted.

ichpen · May 9, 2022

SatGuyScott said:
Heads up... wile trying to grab the update from your server.

Whenever I log in I get the following error from your server.

Forbidden (403)
CSRF verification failed. Request aborted.

Yeah been happening to me for about 8 months, logout/in again and it should work.

mazzly · May 9, 2022

SatGuyScott said:
Heads up... wile trying to grab the update from your server.

Whenever I log in I get the following error from your server.

Forbidden (403)
CSRF verification failed. Request aborted.

That usually means that the csrf token from the login page has expired.. Try reloading the login page before logging in and it should be fine

egerci · May 9, 2022

Hello,

I am using the latest versions of AMPXF and Siropu Ads Manager.
I also use Google ad manager codes in Siropu Ads manager. I need to add the main category Id to the targeting json section in the GAM code.
Sample :

JSON:

json='{"targeting":{"site_div":["AMP"],"categoryId":["xxx"]}}'

Normally, I can pull the main category ID from the $breadcumb variable (inside the PAGE_CONTAINER template)

HTML:

<xf:if is="$breadcrumbs">
    <xf:foreach loop="$breadcrumbs" value="$catarray" i="$i">
        <xf:if is="$catarray.attributes.node_id">
            <xf:set var="{$cat_id_ads.{$i}}" value="{$catarray.attributes.node_id}" />
            <xf:set var="{$cat_name.{$i}}" value="{$catarray.value}" />
        </xf:if>
    </xf:foreach>
</xf:if>

I can't use $breadcrumbs variable in AMP_CONTAINER template.

My question is: is there any variable which I can get the main category id of the thread? If not, how do we access a variable ourselves from within the amp templates? For example to the $breadcrumbs variable.

thank you.

mazzly · May 10, 2022

egerci said:
Hello,

I am using the latest versions of AMPXF and Siropu Ads Manager.
I also use Google ad manager codes in Siropu Ads manager. I need to add the main category Id to the targeting json section in the GAM code.
Sample :

JSON:

json='{"targeting":{"site_div":["AMP"],"categoryId":["xxx"]}}'

Normally, I can pull the main category ID from the $breadcumb variable (inside the PAGE_CONTAINER template)

HTML:

<xf:if is="$breadcrumbs"> <xf:foreach loop="$breadcrumbs" value="$catarray" i="$i"> <xf:if is="$catarray.attributes.node_id"> <xf:set var="{$cat_id_ads.{$i}}" value="{$catarray.attributes.node_id}" /> <xf:set var="{$cat_name.{$i}}" value="{$catarray.value}" /> </xf:if> </xf:foreach> </xf:if>

I can't use $breadcrumbs variable in AMP_CONTAINER template.

My question is: is there any variable which I can get the main category id of the thread? If not, how do we access a variable ourselves from within the amp templates? For example to the $breadcrumbs variable.

thank you.

With "main category id", does that mean the first/leftmost id in breadcrumbs list? i.e. not the current form id?

I'll have to look a bit further into this, but it should be possible for you to add the same "breadcrumbs"-snippet from PAGE_CONTAINER to AMP_CONTAINER and it should likely work the same..

If you only need access to the current forum node (i.e. rightmost in breadcrumbs) then you should be able to use e.g. $__globals.forum.node_id

egerci · May 10, 2022

Hello,

mazzly said:
With "main category id", does that mean the first/leftmost id in breadcrumbs list? i.e. not the current form id?

yes i want exactly that

mazzly said:
I'll have to look a bit further into this, but it should be possible for you to add the same "breadcrumbs"-snippet from PAGE_CONTAINER to AMP_CONTAINER and it should likely work the same..

Actually, i have already tried this. But it didn't work. Maybe I did something wrong. I will try again today

mazzly said:
If you only need access to the current forum node (i.e. rightmost in breadcrumbs) then you should be able to use e.g. $__globals.forum.node_id

I need to target the main category that you have explained above, or an array with all the category / forum node ids. The code I normally use in the PAGE CONTAINER template assigns the entire hierarchy of the category node ID to an array variable. But I have never needed such a detailed targeting.

mazzly · May 10, 2022

egerci said:
Actually, i have already tried this. But it didn't work. Maybe I did something wrong. I will try again today

I had to open the AMP_CONTAINER to check and the breadcrumbs code and macros are the same in both AMP_CONTAINER and PAGE_CONTAINER..

I also tried opening a random AMP page on your forum, and the breadcrumbs render, so the $breadcrumbs is at least available in the AMP_CONTAINER-template context

I'm wondering if there is something (maybe Siropu addon?) that doesn't pass on all needed things to the ad-rendering context?

- You said this same thing works fine on the normal pages with Siropu addon?

egerci · May 10, 2022

mazzly said:
I had to open the AMP_CONTAINER to check and the breadcrumbs code and macros are the same in both AMP_CONTAINER and PAGE_CONTAINER..

I also tried opening a random AMP page on your forum, and the breadcrumbs render, so the $breadcrumbs is at least available in the AMP_CONTAINER-template context

I'm wondering if there is something (maybe Siropu addon?) that doesn't pass on all needed things to the ad-rendering context?
- You said this same thing works fine on the normal pages with Siropu addon?

Hi @mazzly ,
You are right. I have just tried below code in AMP_CONTAINER and render. And it works great.

Code:

        <xf:if is="$breadcrumbs">
            <xf:foreach loop="$breadcrumbs" value="$catarray"  i="$i">
                <xf:if is="$catarray.attributes.node_id">
                    <xf:set var="{$cat_id_ads.{$i}}" value="{$catarray.attributes.node_id}" />
                </xf:if>
            </xf:foreach>
            <xf:set var="$mainCategory" value="{$cat_id_ads|first}" />
        </xf:if>

And yes I can't
yes, you are right. Problem I can't access the variable I defined in the amp_container template from the ad rendering code in the siropu addon.

I need to research this problem a bit in the sirop addon discussion group. Thank you for your help.

egerci · May 10, 2022

mazzly said:
- You said this same thing works fine on the normal pages with Siropu addon?

By the way, to target category id's to GAM code, I use javascript without using siropu addon. As you know it is not possible to use in AMP pages

webroxau · May 11, 2022

@mazzly any way to increase the speed of checks on my site on your end?

mazzly · May 13, 2022

graham_w said:
@mazzly any way to increase the speed of checks on my site on your end?

Not at the moment no.. Is there some specific reason for this?
Google Search Console will also find AMP page issues in bulk but takes much longer to verify when they have been actually fixed.. Wondering if we could enable/allow uploading of the CSVs that can be exported from Search Console to get those same pages directly into the AMPXF problem page list for faster checking of fixed problems with the AMPBot?

JamesBrown · May 24, 2022

I paid for a large forum, and while it proved to be a great busted link checker, that was the only benefit I noted. Won't renew and now disabled the add on. Thanks anyway

diretur · May 30, 2022

Hi @mazzly
I don't know if you've already fixed this, but when a forum is set as a "question" forum, the amp layout isn't working properly (you can check it out here)

mazzly · May 30, 2022

diretur said:
Hi @mazzly
I don't know if you've already fixed this, but when a forum is set as a "question" forum, the amp layout isn't working properly (you can check it out here)

View attachment 268956

I'll have a look at it and release a fix if I find some problem. Thanks for reporting this

mazzly · May 30, 2022

diretur said:
Hi @mazzly
I don't know if you've already fixed this, but when a forum is set as a "question" forum, the amp layout isn't working properly (you can check it out here)

View attachment 268956

If you are talking about the answers/posts not having spacing between each, the difference between AMP and canonical is that the `samAlignCenter"-class has margin: 10px auto;, while the b-container-ad-class doesn't have that..

The fix for that should be to edit your amp_extra.less to have the following:

Code:

.b-container.b-container-ad {
    text-align: center;
    background: transparent;
    border: none;
    margin: 10px auto;
}

Hope that helps.

Cheers
//Jonathan

Ivancas · Aug 2, 2022

JamesBrown said:
I paid for a large forum, and while it proved to be a great busted link checker, that was the only benefit I noted. Won't renew and now disabled the add on. Thanks anyway

how long did you keep it installed?

Recep Baltaş · Aug 2, 2022

JamesBrown said:
I paid for a large forum, and while it proved to be a great busted link checker, that was the only benefit I noted. Won't renew and now disabled the add on. Thanks anyway

How large?

This is our website growing every month.

JamesBrown · Aug 3, 2022

Ivancas said:
how long did you keep it installed?

One Year

Ivancas · Aug 3, 2022

JamesBrown said:
One Year

May you please share your GA stats?

AMPXF - AMP for Xenforo 2 [Paid] 1.4.9

Well-known member

Active member

Forbidden (403)​

Well-known member

Forbidden (403)​

Well-known member

Forbidden (403)​

Member

Well-known member

Member

Well-known member

Member

Member

Active member

Well-known member

Well-known member

Active member

Well-known member

Well-known member

Well-known member

Well-known member

Well-known member

Well-known member

Similar threads

We value your privacy

Forbidden (403)

Forbidden (403)

Forbidden (403)