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

Strip the mobile subdomain? #542

Open
TheSonAlsoRises opened this issue Jun 12, 2016 · 1 comment
Open

Strip the mobile subdomain? #542

TheSonAlsoRises opened this issue Jun 12, 2016 · 1 comment
Labels
enhancement

Comments

@TheSonAlsoRises
Copy link
Contributor

Some URLs are submitted as:

https://m.reddit.com/r/pokemontrades/comments/4mlmo8/ft_56_iv_axewfennekinfroakie_w_ha_ems/d3wfugl?context=3

I am guessing the presence of the mobile subdomain does not allow for trade matching, so maybe we should remove the m. from the URL?
@not-an-aardvark
Copy link
Contributor

Actually, we only check the portion of the URL starting with /r/subreddit to see whether trades match, so the mobile subdomain shouldn't affect it.

However, it can be kind of annoying to click on a mobile link and get redirected to the mobile site, so maybe it's worth changing this for that reason?

@AlMcKinlay AlMcKinlay removed the next label Jun 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@AlMcKinlay @TheSonAlsoRises @not-an-aardvark