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

WIP: tackle Issue #72 (Django 1.10 Compatibility) #74

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

morvan
Copy link

@morvan morvan commented Aug 31, 2016

I added a Django 1.10 support by fetching the loaders in TEMPLATES instead of TEMPLATE_LOADERS.

I now have problems while running the tests:
test_ipad and test_motorola_xoom are failing because for a reason I couldn't identify, the Flavour returned is None.

Any clue about the reason why it happens ?

@morvan morvan changed the title tackle Issue #72 (Django 1.10 Compatibility) WIP: tackle Issue #72 (Django 1.10 Compatibility) Aug 31, 2016
@karolyi
Copy link

karolyi commented Oct 5, 2016

can you do an iteration on this and fix the tests, so @gregmuellegger can merge it?

@karolyi
Copy link

karolyi commented Oct 7, 2016

ref-ing #72, there is additional information about jinja.

@mcrowson
Copy link

Would love for this to get through. Not sure what's left for it.

@urtzai
Copy link

urtzai commented Mar 8, 2017

I need this merge too...

@urtzai
Copy link

urtzai commented Mar 20, 2017

@@ -158,7 +158,7 @@ def test_functional(self):
result = result.strip()
self.assertEqual(result, 'Hello .')
# simulate RequestContext
result = render_to_string('index.html', context_instance=RequestContext(Mock()))
result = render_to_string('index.html', context = RequestContext(Mock()).flatten())

Choose a reason for hiding this comment

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

space before and after the =
context=RequestContext(Mock())

@dokterbob dokterbob mentioned this pull request Apr 23, 2018
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