AMP validation error

The validation error occurs if the wp forms shortcode is in the <amp-mega-menu> component.

I apologize for opening multiple threads.
I removed the following part in “class-frontend.php”:

if ( wpforms_is_amp() ) {

  $state = array(
    'submitting' => false,
  );

  printf(
    '<amp-state id="%s"><script type="application/json">%s</script></amp-state>',
    $this->get_form_amp_state_id( $form_id ),
    wp_json_encode( $state )
  );

}

Will my wpform work without those lines? And if not, can I add amp-state after closing <amp-mega-menu> manually as static?

• This reply was modified 3 years, 1 month ago by egornmore.
• This reply was modified 3 years, 1 month ago by egornmore.
• This reply was modified 3 years, 1 month ago by egornmore.
• This reply was modified 3 years, 1 month ago by egornmore.

Hello @egornmore,

Thanks for writing in.

When you get the chance, can you please let us know which AMP plugin are you using? Currently, WPForms Lite should work with the official Google AMP plugin for WordPress.

Thanks!

Hello @jadealombro,
I am using the official Google AMP plugin By AMP Project Contributors.

Hello @egornmore,

Thanks for confirming.

While it workaround you took solved the issue, it isn’t something we recommend since editing the core files of any plugin would get wiped out when you upgrade the plugin version.

But based on the situation you described, did you get any chance if a form element can be placed inside <amp-mega-menu> as it might be more recommended to place forms inside the <amp-sidebar>.

I might also help if you get in touch with the official Google AMP plugin support for this as this should be something they can confirm.

Thank you.

Hi @egornmore,

We haven’t heard back from you in about a week, so I’m going to go ahead and close this thread for now. But if you’d like us to assist further, please feel welcome to continue the conversation.

Thanks!

Hi @jadealombro

I’ll try to play around and use <amp-sidebar> instead.

Thanks for your time!