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.

Sign up for GitHub

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

Jump to bottom

A smooth way to migrate from other AMP plugins to AMPforWP #1051

Closed
MARQAS opened this issue Aug 16, 2017 · 1 comment
Closed

A smooth way to migrate from other AMP plugins to AMPforWP #1051

MARQAS opened this issue Aug 16, 2017 · 1 comment
Assignees
@ajeetku
Milestone
0.9.63

Comments

@MARQAS
Copy link
Contributor

MARQAS commented Aug 16, 2017

There are many users who want to migrate from other AMP plugins to this and they want to redirect those URLs to the correct amphtml i.e., /amp/ in the end.

reference: https://wordpress.org/support/topic/go-from-site-comamppost-to-site-compostamp/

This is just one example, much more are there. If we can give them a better solution, they will not hesitate much about their URLs.
@Frenchomatic
Copy link

Redirectmatch will probably be the most efficient way to do this.

@ahmedkaludi ahmedkaludi modified the milestone: 0.9.60 Aug 25, 2017
@ahmedkaludi ahmedkaludi modified the milestones: 0.9.60, 0.9.61 Sep 4, 2017
@ahmedkaludi ahmedkaludi modified the milestones: 0.9.61, 0.9.63 Sep 26, 2017
ajeetku added a commit that referenced this issue Sep 29, 2017
@ajeetku
#1051,#1079
1fc1963 
amp-category-base-remover added

Added Alert Dependency OLD AMP installation
ajeetku added a commit that referenced this issue Sep 29, 2017
@ajeetku
#1051
eb667cb 
Removed amp from inside amp

exclude the pagination
if there is page is added than, current code will remove the amp

 1. if there is amp in current URL
 2. if page is not found in current url
ajeetku added a commit that referenced this issue Oct 6, 2017
@ajeetku
#1051
6f2cbdc 
removed undefine index issue
ajeetku added a commit that referenced this issue Oct 6, 2017
@ajeetku
#1051 alert on installation of OLD AMP after search
aecf84d 
#1051 alert on installation of OLD AMP after search
MohammedKaludi added a commit that referenced this issue Oct 7, 2017
@MohammedKaludi
updated the alter text #1051
c9747e9 
updated the alter text #1051
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ajeetku ajeetku

Labels
None yet
Projects
None yet
Milestone
0.9.63
Development

No branches or pull requests
4 participants
@Frenchomatic @MARQAS @ahmedkaludi @ajeetku