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

Allow {% call %} blocks to access parent scope. Fixes #906 #994

Merged
merged 1 commit into from
May 24, 2017

Conversation

fdintino
Copy link
Collaborator

Summary

Proposed change:

Allows access to parent scope inside {% call %} tags.

Closes #960 .

Checklist

I've completed the checklist below to ensure I didn't forget anything. This makes reviewing this PR as easy as possible for the maintainers. And it gets this change released as soon as possible.

@jgerigmeyer
Copy link
Contributor

Wonderful! It's worth noting that though they're related, this does not fix #912 (failing tests added in #917), but it's definitely an improvement. Thanks!

@ArmorDarks
Copy link

Thanks for your hard work!

@fdintino
Copy link
Collaborator Author

@jgerigmeyer yes, this is just a band-aid. The management of template symbols and scope resolution needs a fair bit of reworking to really address these issues.

@fdintino fdintino merged commit efd2ee9 into master May 24, 2017
@fdintino fdintino deleted the fix-call-scope branch January 17, 2018 06:00
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.

Using Nunjucks in Flask application - including HTML
3 participants