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

creole markup rendering on github is broken, leaving 404 urls #8

Open
MetaMemoryT opened this issue Aug 21, 2015 · 4 comments
Open
Assignees

Comments

@MetaMemoryT
Copy link

Some links like the one in Readme.creole:

Metatrader using OTMql4Py (https://github.com/OpenTrading/OTMql4Py). 

Leave the closing ) as part of the url. Then the url

https://github.com/OpenTrading/OTMql4Py)

Is 404,

Could we switch to an alternate creole url syntax or leave a space after url end to mitigate the broken url issue?

@femtotrader
Copy link

Maybe you might write an issue here https://github.com/github/markup/issues

@OpenTrading
Copy link
Owner

Github's creole is really broken: the rendering of creole in the wiki is different from the rendering of the top-level README.creole. I have some work arounds for the README that I've done another pass on, so it should be better now, but it really needs fixing at their end.

Thanks femtotrader for the link - I was looking for the repository for the creole rendering so I could take a look at the code. I'll look again for it or if you find it please add it here. The problem is, I can't see why the rendering is different (better) in the wiki than the top-level!

@OpenTrading
Copy link
Owner

A trailing space after the link works around it - they all should be OK now. The page won't change much now.

The README.creole file is a copy of FrontPage.creole in the wiki, with a sed script to try to work around the brokeness. In the wiki, things work OK, even though their implementation is lame.

Thank you for your help catching these details as it makes the project look bad with broken URLs.

@OpenTrading OpenTrading self-assigned this Aug 30, 2015
@OpenTrading
Copy link
Owner

I think these are all fixed, but I'll leave the issue open to remind me to track down a simple case to submit a bug report to github.

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

No branches or pull requests

3 participants