Skip to content
This repository has been archived by the owner on Oct 10, 2018. It is now read-only.

Add filter for AMPNews plugin path #92

Open
wants to merge 54 commits into
base: develop
Choose a base branch
from

Conversation

davisshaver
Copy link

No description provided.

@westonruter
Copy link
Contributor

Why is this needed?

@davisshaver
Copy link
Author

@westonruter if you use composer to manage AMP-WP, the path might be amp-wp/amp.php for example. This prevents the nag from always being there, and improves the version check if there happens to be a version loaded already that could be a higher number.

@westonruter
Copy link
Contributor

I think you would probably be better of to just make these changes to your own fork. I don't foresee this theme being maintained further where it is right now as it was really made for the purpose of being a demo for AMP Conf.

@westonruter westonruter mentioned this pull request Sep 27, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants