Skip to content

package name was wrong in docs and in exception message#41

Open
il-ja wants to merge 1 commit into
TyMaszWeb:masterfrom
il-ja:master
Open

package name was wrong in docs and in exception message#41
il-ja wants to merge 1 commit into
TyMaszWeb:masterfrom
il-ja:master

Conversation

@il-ja
Copy link
Copy Markdown

@il-ja il-ja commented Aug 3, 2018

It's django.template.context_processors.request -- not ...core...

this was correct in the context_processors-list in the readme, but wrong in the line above. Also, it was wrong in the the error-message thrown by the templatetag.

PS: thank you for the package :)

 - not .core. instead of .template.
 - changed it in the docs and in the error message
Copy link
Copy Markdown

@tpaljor tpaljor left a comment

Choose a reason for hiding this comment

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

Documentation is correct in this PR - verified working with Django==3.0.6

@piotrkilczuk
Copy link
Copy Markdown
Member

When did the core package get removed @tpaljor ?

There's a long way from Django 1.8 to 3.0...

@piotrkilczuk piotrkilczuk force-pushed the master branch 2 times, most recently from d524568 to 5da5c54 Compare January 13, 2021 14:11
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.

3 participants