Skip to content
This repository was archived by the owner on Nov 25, 2017. It is now read-only.

Changed .html to .rst to fix links - #96

Open
nqobile wants to merge 1 commit into
jacobian:masterfrom
nqobile:patch-20
Open

Changed .html to .rst to fix links#96
nqobile wants to merge 1 commit into
jacobian:masterfrom
nqobile:patch-20

Conversation

@nqobile

@nqobile nqobile commented Jun 3, 2014

Copy link
Copy Markdown

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.

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.
@treyhunner

Copy link
Copy Markdown

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 git@github.com: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.

@nqobile

nqobile commented Jun 5, 2014

Copy link
Copy Markdown
Author

@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.

@suhailvs

Copy link
Copy Markdown

@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.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants