Accelerated-mobile-pages: Problem with Disqus
Great work integrating Disqus, means that my own solution might not be needed very soon, however, there is a problem that I just noticed.
Disqus on AMP isn't pulling or pushing to the original article URL, this probably because it has been set to the URL of the AMP version of the article, not the original, however, I am unsure as I haven't had time to go through the code.
nicholasgriffintn
>All comments
I've figured out the reason why.
We have a character in our article URLs, specifically an "=", the php that you guys are using in includes/disqus.php to remove the AMP tag from the URL is also removing this special character, and so it's not loading the correct links.
Something that I can probably fix with my own loaded code, which is what I will be doing anyway as I prefer to load from my own URLs, however, I figured that letting you know would be handy in case anyone else has the issue.
nicholasgriffintn
on 19 Feb 2017
Related issues
jeherve
路
8Comments
Blackburn789
路
4Comments
MohammedKaludi
路
8Comments
HaripriyaHari
路
7Comments
ghost
路
4Comments
Most helpful comment
I've figured out the reason why.
We have a character in our article URLs, specifically an "=", the php that you guys are using in includes/disqus.php to remove the AMP tag from the URL is also removing this special character, and so it's not loading the correct links.
Something that I can probably fix with my own loaded code, which is what I will be doing anyway as I prefer to load from my own URLs, however, I figured that letting you know would be handy in case anyone else has the issue.