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.

Sign up for GitHub

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

Jump to bottom

Actually use the Jinja2 template backend #1882

Merged
merged 2 commits into from
Jul 5, 2024
Merged

Actually use the Jinja2 template backend #1882

merged 2 commits into from
Jul 5, 2024

Commits on Jul 5, 2024

  1. Instrument the Django Jinja2 template backend. 

    - Add jinja2 template to example app.
- Switch to the render function to include context.

It instruments the single template render, but not the inherited
templates and I'm guessing not the included templates either.
I suspect we're going to have to patch jinja templates more robustly
than relying on the django jinja backend template class.

Co-Authored-By: Tim Schilling <schillingt@better-simple.com>
    @matthiask @tim-schilling
    matthiask and tim-schilling committed Jul 5, 2024
    Configuration menu
    Copy the full SHA
    c8f6763 View commit details
    Browse the repository at this point in the history

  2. Improve the jinja tests to better indicate the situation.

    @tim-schilling
    tim-schilling committed Jul 5, 2024
    Configuration menu
    Copy the full SHA
    4fd886b View commit details
    Browse the repository at this point in the history