Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Notice errors after activating amp-woocommerce plugin. #3054

Closed
syedimran22 opened this issue Apr 19, 2019 · 3 comments
Closed

Notice errors after activating amp-woocommerce plugin. #3054

syedimran22 opened this issue Apr 19, 2019 · 3 comments
Assignees
Milestone

Comments

@syedimran22
Copy link
Contributor

Notice: Undefined index: amp-woocommerce-pro in C:\xampp\htdocs\wordpress\wp-content\plugins\accelerated-mobile-pages\includes\options\admin-config.php on line 532

@MARQAS MARQAS added the bug label Apr 23, 2019
@MARQAS MARQAS added this to the Wave milestone Apr 23, 2019
@pegazee pegazee self-assigned this Apr 23, 2019
@pegazee
Copy link
Contributor

pegazee commented Apr 23, 2019

Resolved code is already added in the beta 8 months ago. And I've tried to recollect the ticket but didn't find

Note:

When the merging the ticket in the master, make sure to add the below code:

$license_status = ''; if(isset($selectedOption['amp-license'][$pathExploded]['status']) && $selectedOption['amp-license'][$pathExploded]['status']==='valid'){ $license_status = $selectedOption['amp-license'][$pathExploded]['status']; $currentStatus = 'active valid';
.
.
.
.
.
.
<input name="redux_builder_amp[amp-license]['.$pathExploded.'][status]" type="hidden" value="'.$license_status.'">';

@pegazee
Copy link
Contributor

pegazee commented Apr 25, 2019

** Review **

  • Kaleem

  • Imran

pegazee added a commit that referenced this issue Apr 25, 2019
@pegazee
Copy link
Contributor

pegazee commented Apr 26, 2019

Pushed in 0.9.97.49

@pegazee pegazee closed this as completed Apr 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants