This repository has been archived by the owner. It is now read-only.

Update chapter05.rst #33

Merged
merged 1 commit into from Apr 14, 2013

Conversation

Projects
None yet
3 participants
@tariwan
Contributor

tariwan commented Feb 21, 2013

new Django syntax in INSTALLED_APPS(settings.py)

Update chapter05.rst
new Django syntax in INSTALLED_APPS(settings.py)

jacobian added a commit that referenced this pull request Apr 14, 2013

Merge pull request #33 from tariwan/patch-1
Update chapter05.rst

Conflicts:
	chapter05.rst

@jacobian jacobian merged commit 9e97dfb into jacobian:master Apr 14, 2013

@tariwan tariwan deleted the tariwan:patch-1 branch Apr 15, 2013

@guelo

This comment has been minimized.

Show comment Hide comment
@guelo

guelo Jul 6, 2013

The change from 'mysite.books' to 'books' broke this section for me, I'm going through the book using django 1.4. If this change is some new syntax in 1.5 then maybe it should be explained? Though Chapter 1 says the book is for 1.4.

Also, the paragraph at line 578 still mentions 'mysite.books' and contradicts the 'books' usage since it says you need the full python path:
```'mysite.books'refers to thebooks`` app we're working on. Each app in
``INSTALLED_APPS`` is represented by its full Python path -- that is, the path
of packages, separated by dots, leading to the app package.`

guelo commented Jul 6, 2013

The change from 'mysite.books' to 'books' broke this section for me, I'm going through the book using django 1.4. If this change is some new syntax in 1.5 then maybe it should be explained? Though Chapter 1 says the book is for 1.4.

Also, the paragraph at line 578 still mentions 'mysite.books' and contradicts the 'books' usage since it says you need the full python path:
```'mysite.books'refers to thebooks`` app we're working on. Each app in
``INSTALLED_APPS`` is represented by its full Python path -- that is, the path
of packages, separated by dots, leading to the app package.`

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