OTMql4Zmq icon indicating copy to clipboard operation
OTMql4Zmq copied to clipboard

creole markup rendering on github is broken, leaving 404 urls

Open MetaMemoryT opened this issue 9 years ago • 4 comments

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?

MetaMemoryT avatar Aug 21 '15 16:08 MetaMemoryT

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

femtotrader avatar Aug 21 '15 19:08 femtotrader

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 avatar Aug 22 '15 03:08 OpenTrading

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 avatar Aug 22 '15 03:08 OpenTrading

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.

OpenTrading avatar Aug 30 '15 14:08 OpenTrading