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

use first pass output for jinja templates whenever possible #153

Merged
merged 1 commit into from
Sep 19, 2023

Conversation

sHermanGriffiths
Copy link
Contributor

@sHermanGriffiths sHermanGriffiths commented Sep 19, 2023

Describe Your Changes

  • wrapped render_content filter in _debug_assist
  • delete unnecessary reference to template.root_render_func
  • don't rerender templates that don't need to be

How Did You Test It

I exported a Prenuvo document (see Figures 1 and 2) that used both types of jinja templates and ensured that both types of templates rendered properly

Figure 1

Screen Shot 2023-09-19 at 10 20 29 AM

Figure 2

Screen Shot 2023-09-19 at 10 20 59 AM

@sHermanGriffiths sHermanGriffiths marked this pull request as ready for review September 19, 2023 15:23
@sHermanGriffiths sHermanGriffiths merged commit 548bd71 into main Sep 19, 2023
3 checks passed
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