Ivan Privalov

The
<link rel=”shortcut icon” href=”https://www.yourwebsite.com/wp-content/uploads/someimage.png” title=”Favicon”>

will not validate!

Great.

Thank you.

This is great.
Can you please add this code example to the Documentation of the Custom Theme?

Thank you.

Great.

p.s. I am not complaining at all. I am more than thankful for the great plugin and would like to help with the above debugging ??

@ampforwp any page that does not have the
<amp-accordion><section>… will cause this validation error.

p.s. I am currently using your Custom Theme plugin.

Thank you, Ryan.

If your drop-down menu does not have child elements, you can remove amp-accordion library using the following code in the child-theme:

// Remove AMP-accordion Library
add_filter( ‘amp_post_template_data’, ‘gp_ampforwp_add_design3_required_scripts’, 101 );

function gp_ampforwp_add_design3_required_scripts( $data ) {
unset($data[‘amp_component_scripts’][‘amp-accordion’]);
return $data;
}

I still get this issue with the Custom Theme.
Perhaps you should wrap the menu into the <amp-accordion>?

Warning quote:
The extension ‘amp-accordion extension .js script’ was found on this page, but is unused (no ‘amp-accordion’ tag seen). This may become an error in the future.

Hello Ahmed, please see my above solution.
It perfectly worked for me.

$page_for_posts in …accelerated-moblie-pages.php on line 39 returns ‘0’ outside of the Loop all the way in the top of the pages [including Homepage].

The function in …accelerated-moblie-pages.php on line 39 needs to be changed to:
function ampforwp_name_blog_page() {
if(!$page_for_posts = get_option( ‘page_for_posts’ )) return;
$post = get_post($page_for_posts);
$slug = $post->post_name;
return $slug;
}

p.s. You have several non-closed shortcodes in the bottom of https://ampforwp.com/story/?ThanksUser

Screenshot: https://img02.imgland.net/KuQ3QKf.png

Same issue in the frontend:
Trying to get property of non-object in …accelerated-moblie-pages.php on line 39

The installation process ended with the following error:
Notice: Trying to get property of non-object in \www\blog\wp-content\plugins\accelerated-mobile-pages\accelerated-moblie-pages.php on line 39

Warning: Cannot modify header information – headers already sent by (output started at \www\blog\wp-content\plugins\accelerated-mobile-pages\accelerated-moblie-pages.php:39) in \www\blog\wp-includes\pluggable.php on line 1195

Warning: Cannot modify header information – headers already sent by (output started at \www\blog\wp-content\plugins\accelerated-mobile-pages\accelerated-moblie-pages.php:39) in \www\blog\wp-includes\pluggable.php on line 1195

The WordPress requires the BETWEEN values to be sent as an array.

I therefore modified the call to:
&csrp_meta_key=price&csrp_meta_compare=BETWEEN&csrp_meta_value=100,500&csrp_meta_type=ARRAY

and adjusted the custom-simple-rss.php code:

	if( isset($_GET["csrp_meta_type"]) ){
		$csrp_meta_type = sanitize_text_field($_GET["csrp_meta_type"]);	
	}else{
		$csrp_meta_type = 'CHAR';	
	}

TO:

	if( isset($_GET["csrp_meta_type"]) ){
		$csrp_meta_type = sanitize_text_field($_GET["csrp_meta_type"]);
		if($csrp_meta_type == "ARRAY") {
			$csrp_meta_type		= 'numeric';
			$csrp_meta_value	= explode(',', $csrp_meta_value);
		}
	}else{
		$csrp_meta_type = 'CHAR';
	}

Another option I would love is to have an option to choose the back-end type of map – roadmap/satellite…