djangobook.com icon indicating copy to clipboard operation
djangobook.com copied to clipboard

Changed .html to .rst to fix links

Open nqobile opened this issue 11 years ago • 3 comments

Fixed the links at the end of the page that link to Chapter 14 and Chapter 16. The links are actually jacobian/djangobook.com/blob/master/chapter14.rst and jacobian/djangobook.com/blob/master/chapter16.rst not ../chapter14.html and ../chapter16.html, respectively.

nqobile avatar Jun 03 '14 01:06 nqobile

Hey @nqobile!

Disclaimer: I am not one of the maintainers of this project. If @jacobian or one of the other maintainers wants to step in and correct or verify my interpretation, please do so.

From my understanding pull requests #77 through #96 cannot be accepted by this project. This is because the djangobook.com project files are used to generate the website http://www.djangobook.com/ and are primarily meant to be read from the djangoproject.com website and not from Github. Your changes seem to introduce a book on that website.

Here's the process I used to verify that your changes won't work out for the djangoproject.com website:

  1. I cloned your repository and checked out one of your pull requests to try to build the project locally:

    git clone [email protected]:nqobile/djangobook.com.git
    cd djangobook.com
    git checkout patch-6
    
  2. I built the project locally:

    sudo pip install Sphinx  # I didn't have Sphinx installed
    make html
    
  3. I opened the file _build/html/chapter06.html in Chrome and clicked on the Chapter 7 link at the bottom of the page. The link took me to chapter07.rst (which gave me a 404 error) instead of chapter07.html.

Thanks for trying your hand at contributing. I'm sorry no one said anything a few hours ago while you were in the process of making your contributions. I was away from my computer at the time.

treyhunner avatar Jun 03 '14 04:06 treyhunner

@treyhunner

That makes sense. I was using djangobook.com trying to understand Django 1.6, but their disclaimer on the page led me to think that they were now pointing people to this repository for newer versions of Django. Since their disclaimer is still on the website I didn't think that they were using these files to generate the content for the book.

nqobile avatar Jun 05 '14 21:06 nqobile

@nqobile yes they were using these files to generate the content of djangobook.com. it is by sphinx. please see my fork https://github.com/suhailvs/djangobook.com. you can see its sphinx generated pages at http://suhailvs.github.io/djangobook.com/. I am planning to maintain the book, since it is a great book and helped me alot.. so feel free to fork mine and send me pull requests.

suhailvs avatar Jun 16 '14 01:06 suhailvs