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

[#37] - Added simple home page, templates folder and modify settings and... #39

Merged
merged 3 commits into from
Apr 1, 2015

Conversation

dorotawm
Copy link
Contributor

... main urls

TEMPLATE_DIRS = (os.path.join(BASE_DIR, 'accounts/templates'), )
TEMPLATE_DIRS = (os.path.join(BASE_DIR, 'accounts/templates'),
os.path.join(BASE_DIR, 'templates')
)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

a co z places/templates ?
ps. skoro to trzeba dodać to jakim cudem to wrześniej działało ?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nie trzeba tego dodawać, wystarczy dodać aplikację do INSTALLED_APPS i samo będzie znajdywało szablony w folderze "templates" w aplikacji.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To jest zewnętrzny katalog, dodany poza aplikację, dlatego trzeba to dodać inaczej nie przeszukuje tego katalogu (nie ma aplikacji) to jest luźny katalog.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No ale mówimy tutaj przecież o "accounts/templates", a accounts to jest przecież aplikacja.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Aaaa, ok, no tak to jest niepotrzebne, ale to już było dodane wcześniej. Można to usunąć i zostawić sam wpis z templates. 👍

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dzięki :)

@lukaszjagodzinski
Copy link
Contributor

👍


urlpatterns = patterns('',
url(r'^admin/', include(admin.site.urls)),
url(r'^$', TemplateView.as_view(template_name='index.html'), name="home"),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

raz apostrofy, raz cudzysłowy - zdecyduj się :)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

zdecydowałam się... chyba mamy tu więcej komentarzy niż linii kodu ;)

lukaszjagodzinski pushed a commit that referenced this pull request Apr 1, 2015
[#37] - Added simple home page, templates folder and modify settings and...
@lukaszjagodzinski lukaszjagodzinski merged commit b3649fd into master Apr 1, 2015
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

Successfully merging this pull request may close these issues.

5 participants