Skip to content

Conversation

@FarhanAliRaza
Copy link

Related to #557

Copy link
Owner

@adamchainz adamchainz left a comment

Choose a reason for hiding this comment

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

Thanks for working on this, it's shaping up well. I have these suggestions.

@@ -1,3 +1,2 @@
django
django-template-partials
Copy link
Owner

Choose a reason for hiding this comment

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

Did you test the example project with a fresh venv without django-template-partials installed? I think we need to drop template_partials from INSTALLED_APPS, at least:

"template_partials",

Copy link
Author

Choose a reason for hiding this comment

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

No, I just ran manage.py check command.
I'll run and check.

Comment on lines -44 to -45
Using django-template-partials
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Copy link
Owner

Choose a reason for hiding this comment

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

Can we title this section "Using template partials" and then explain in terms of both django-template-partials and Django 6.0's new feature? I want it to still work for users who haven't upgraded to 6.0, especially since it's not even out yet!

Perhaps explain the 6.0 way, then the differences in django-template-partials: install the app, use {% load partial %} as well, ...

@adamchainz adamchainz changed the title added django6.0 template-partials and removed django-template-partials Update partial rendering docs to cover Django 6.0's built-in support Sep 25, 2025
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.

2 participants