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

Fixed #20995 -- Added support for iterables of template names to {% include %} template tag. #12407

Merged
merged 1 commit into from
Feb 18, 2020

Conversation

keshav2212
Copy link

No description provided.

django/template/loader_tags.py Outdated Show resolved Hide resolved
django/template/loader_tags.py Outdated Show resolved Hide resolved
@timgraham timgraham removed their request for review February 3, 2020 13:53
docs/ref/templates/builtins.txt Outdated Show resolved Hide resolved
docs/releases/3.1.txt Outdated Show resolved Hide resolved
@keshav2212
Copy link
Author

Is there more changes left ?

@adamchainz
Copy link
Sponsor Member

Not from me but the fellows might spot something. Make sure the ticket on Trac is not marked as needing improvement so that they see it.

@keshav2212
Copy link
Author

Ok, I have unset it.

@felixxm felixxm self-assigned this Feb 17, 2020
@felixxm felixxm changed the title Fixed #20995 -- Improved {% include %} when giving template name list Fixed #20995 -- Added support for iterables to {% include %} template tag. Feb 17, 2020
@felixxm felixxm changed the title Fixed #20995 -- Added support for iterables to {% include %} template tag. Fixed #20995 -- Added support for iterables of template names to {% include %} template tag. Feb 17, 2020
Copy link
Member

@felixxm felixxm left a comment

Choose a reason for hiding this comment

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

@keshav2212 Thanks for this patch 👍 I pushed edits to docs, made tests use subTest(), and added test and fix for None.

docs/ref/templates/builtins.txt Outdated Show resolved Hide resolved
@felixxm felixxm force-pushed the ticket_20995 branch 2 times, most recently from 3b03e0c to 143d87a Compare February 17, 2020 19:05
@keshav2212
Copy link
Author

@felixxm Thank you for review.

…nclude %} template tag.

Thanks Adam Johnson for the review.
@felixxm felixxm merged commit f37d548 into django:master Feb 18, 2020
@keshav2212
Copy link
Author

Thanks @adamchainz , @kezabelle and @felixxm For helping me in my first issue.

@keshav2212 keshav2212 deleted the ticket_20995 branch February 18, 2020 08:44
@felixxm
Copy link
Member

felixxm commented Feb 18, 2020

@keshav2212 Welcome aboard ⛵

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants