Use 'generate_stats' instead of 'process_response'

node13h / django-debug-toolbar-template-profiler

Displays template rendering time on the timeline

Other

72 stars 14 forks source link

Use 'generate_stats' instead of 'process_response' #8

Closed goffioul closed 5 years ago

goffioul commented 5 years ago

This is required for django-debug-toolbar 2.

blueyed commented 5 years ago

..or rather process_request, no?

generate_stats's doc says it might not be called on every request.

Ref: https://github.com/jazzband/django-debug-toolbar/commit/5299cf8 / https://github.com/jazzband/django-debug-toolbar/pull/1140

Penagwin commented 5 years ago

I was only able to get it to work by using generate_stats, I didn't have any luck with your pull request @blueyed

adamchainz commented 5 years ago

@Penagwin I've merged #13 and released as version 2.0.0, because it worked for me in testing as well. What errors were you seeing without using generate_stats?