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

Fix issue #3674 - Second solution #3935

Merged
merged 2 commits into from
Apr 13, 2015

Conversation

yakky
Copy link
Member

@yakky yakky commented Mar 8, 2015

Alternative approach to #3674

Fixes #3674
Fixes #3917

@yakky yakky added this to the 3.0.13 milestone Mar 8, 2015
@yakky yakky changed the title Fix issue #3674 Fix issue #3674 - Second solution Mar 8, 2015
@yakky
Copy link
Member Author

yakky commented Mar 13, 2015

@mkoistinen is this a more appropriate fix than #3917?

try:
instance.rescan_placeholders()
except TemplateDoesNotExist as e:
warnings.warn('Exception occurred: %s template does not exists' % e)
Copy link
Contributor

Choose a reason for hiding this comment

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

'...template does not exist' - remove the final 's'.

@yakky yakky mentioned this pull request Apr 7, 2015
mkoistinen added a commit that referenced this pull request Apr 13, 2015
@mkoistinen mkoistinen merged commit d65426f into django-cms:support/3.0.x Apr 13, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants