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

Add magnet link support #64

Open
xiongchiamiov opened this issue Feb 26, 2015 · 2 comments
Open

Add magnet link support #64

xiongchiamiov opened this issue Feb 26, 2015 · 2 comments

Comments

@xiongchiamiov
Copy link

As noted in this bug report, we don't currently support magnet links.

FWIW, the example CommonMark implementation handles this fine.

Up to you whether this is anything we're interested in supporting.

@jsayol
Copy link

jsayol commented Mar 7, 2015

I think it's worth adding support for magnet links, it adds functionality and should be a quick fix anyway.

Edit: if I'm not mistaken, it would be enough to add 'magnet:' here.
Edit 2: And/or here. First time looking at reddit's source code so I'm a bit lost :)

@jsayol
Copy link

jsayol commented Mar 7, 2015

By the way, here it says "mailto://" when it should be just "mailto:". I did a quick test and mailto links don't work right now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants