Skip to content

Allow recursive extend in rendering#6564

Merged
goutnet merged 7 commits intodjango-cms:developfrom
stefanw:recursive-extend
May 19, 2021
Merged

Allow recursive extend in rendering#6564
goutnet merged 7 commits intodjango-cms:developfrom
stefanw:recursive-extend

Conversation

@stefanw
Copy link
Contributor

@stefanw stefanw commented Nov 22, 2018

Summary

Fixes #6563

Proposed changes in this pull request

Add the ability to extend templates recursively to Django CMS. This is possible in Django since 1.9.

Documentation checklist

  • I have updated CHANGELOG.txt if appropriate
  • I have updated the release notes document if appropriate, with:
    • general notes
    • bug-fixes
    • improvements/new features
    • backwards-incompatible changes
    • required upgrade steps
    • names of contributors
  • I have updated other documentation
  • I have added my name to the AUTHORS file
  • This PR's documentation has been approved by Daniele Procida

@stefanw stefanw changed the base branch from develop to release/3.5.x November 22, 2018 18:04
@stefanw stefanw changed the base branch from release/3.5.x to release/3.6.x November 22, 2018 18:04
@coveralls
Copy link

coveralls commented Nov 22, 2018

Coverage Status

Coverage remained the same at 78.215% when pulling b65e78c on stefanw:recursive-extend into c4b48fa on divio:develop.

@stefanw stefanw changed the title Recursive extend Allow recursive extend in rendering Nov 22, 2018
@stefanw
Copy link
Contributor Author

stefanw commented Nov 26, 2018

At the moment CI on base branch is failing, I will rebase when release/3.6.x passes CI again.

https://docs.djangoproject.com/en/dev/releases/1.9/#template-loader-apis-have-changed
Recursive template extending works since Django 1.9, but it was not
working when rendering placeholders. Adding the extend node origin to
the template in the context allows recursion to work.
@FinalAngel
Copy link
Member

@stefanw would you be able to open the PR against develop?

@stefanw stefanw changed the base branch from release/3.6.x to develop August 7, 2019 09:10
@FinalAngel FinalAngel added this to the 3.8.x milestone Aug 7, 2019
@stefanw
Copy link
Contributor Author

stefanw commented Aug 16, 2019

@FinalAngel I changed the base branch to develop.

@goutnet goutnet merged commit fed6fe5 into django-cms:develop May 19, 2021
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.

Recursive template extending does not work with placeholders

5 participants