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

When Mobile redirection is On,the View Non-AMP link in the footer does not redirect to Non-AMP. #3884

Closed
Arihantkankariya09 opened this issue Nov 28, 2019 · 3 comments
Closed

When Mobile redirection is On,the View Non-AMP link in the footer does not redirect to Non-AMP. #3884

Arihantkankariya09 opened this issue Nov 28, 2019 · 3 comments
Assignees
@WasimM3
Labels
bug NEXT UPDATE [Priority: HIGH]
Milestone
Stabilize 4.0

Comments

@Arihantkankariya09
Copy link

video_2019-11-28_18-09-43.zip

The link to Non-AMP is redirecting to AMP only.

Ref:- https://secure.helpscout.net/conversation/1005961095/97814?folderId=3243584
@MohammedKaludi MohammedKaludi added this to the Matching Up milestone Nov 29, 2019
@Zabi09 Zabi09 self-assigned this Nov 29, 2019
@jasthilokesh jasthilokesh self-assigned this Dec 9, 2019
@ahmedkaludi ahmedkaludi modified the milestones: Matching Up, Stabilize Dec 31, 2019
@WasimM3 WasimM3 self-assigned this Jan 8, 2020
@MohammedKaludi MohammedKaludi assigned Zabi09 and unassigned WasimM3 Apr 4, 2020
@Zabi09 Zabi09 assigned WasimM3 and Zabi09 and unassigned Zabi09 Apr 15, 2020
WasimM3 added a commit that referenced this issue Apr 21, 2020
@WasimM3
#3884 - Fixed the issue
058ea06
@MohammedKaludi
Copy link
Collaborator

if unset is done, why added again?
058ea06#diff-ab576767b896423d184ddafe2ede2310R70

@MohammedKaludi
Copy link
Collaborator

ready to merge.

WasimM3 added a commit that referenced this issue May 15, 2020
@WasimM3
#3884 - Merged in 1.0.50
92f0342
WasimM3 added a commit that referenced this issue May 20, 2020
@WasimM3
#3884 - Added Escaping and merged in 1.0.50
35a6404
WasimM3 added a commit that referenced this issue May 20, 2020
@WasimM3
#3884 - Added Switch and merged in 1.0.50
5c1c954
MohammedKaludi added a commit that referenced this issue May 20, 2020
@MohammedKaludi
#3884 updated the text
cfc20a3 
#3884 updated the text
MohammedKaludi added a commit that referenced this issue May 20, 2020
@MohammedKaludi
#3884 removed double escaping
8bf414c 
#3884 removed double escaping
@MohammedKaludi
Copy link
Collaborator

Fixed and pushed in 1.0.50

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@WasimM3 WasimM3

Labels
bug NEXT UPDATE [Priority: HIGH]
Projects
None yet
Milestone
Stabilize 4.0
Development

No branches or pull requests
6 participants
@jasthilokesh @ahmedkaludi @MohammedKaludi @Arihantkankariya09 @Zabi09 @WasimM3