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

Use of undefined constant AMP_QUERY_VAR #596

Closed
ahmedkaludi opened this issue Feb 19, 2017 · 2 comments
Closed

Use of undefined constant AMP_QUERY_VAR #596

ahmedkaludi opened this issue Feb 19, 2017 · 2 comments
Milestone

Comments

@ahmedkaludi
Copy link
Owner

Today I installed this plugin along with AMP.

The Plugin is great with all the required options. Thanks for the free version.

But I see two issues here.

  1. two notices appear on the home page and admin page

Notice: Use of undefined constant AMP_QUERY_VAR – assumed ‘AMP_QUERY_VAR’ in /home/krazyu2i/public_html/wp-content/plugins/accelerated-mobile-pages/accelerated-moblie-pages.php on line 25
Notice: Use of undefined constant AMP_QUERY_VAR – assumed ‘AMP_QUERY_VAR’ in /home/krazyu2i/public_html/wp-content/plugins/accelerated-mobile-pages/accelerated-moblie-pages.php on line 26

REF: https://wordpress.org/support/topic/use-of-undefined-constant-amp_query_var/

@khaledMohammed000
Copy link
Contributor

done here 5811ba8

MohammedKaludi added a commit that referenced this issue Mar 3, 2017
Added the same filter as the parent plugin as of parent plugin so that
when some one tries to change the endpoint it should work with our
plugin too.
@MohammedKaludi
Copy link
Collaborator

Closing it now.

@ahmedkaludi ahmedkaludi modified the milestones: March 2016, 0.9.44 Mar 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants