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

Debug toolbar enable #3747

Merged
merged 2 commits into from Apr 7, 2022
Merged

Debug toolbar enable #3747

merged 2 commits into from Apr 7, 2022

Conversation

bufke
Copy link
Contributor

@bufke bufke commented Mar 30, 2022

Description

For development, add DEBUG_TOOLBAR environment variable to enable django-debug-toolbar.

Implementation

Currently one must look up their internal IP address as seen by the docker container to enable django debug toolbar. Which is only ever enabled in dev anyway. Instead, we can control this via a environment variable. In dev, set DEBUG_TOOLBAR to "True" to enable it. This does not work in other environments, where debug toolbar isn't enabled at all.

@noliveleger noliveleger self-requested a review April 7, 2022 19:27
@noliveleger noliveleger self-assigned this Apr 7, 2022
Copy link
Contributor

@noliveleger noliveleger left a comment

Choose a reason for hiding this comment

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

LGTM.
Way better than (my 🥴) previous work ;-)

@noliveleger noliveleger merged commit ca2007d into beta Apr 7, 2022
@noliveleger noliveleger deleted the debug-toolbar-enable branch April 7, 2022 19:34
@bufke
Copy link
Contributor Author

bufke commented Apr 7, 2022

I know way back in the day there was no SHOW_TOOLBAR_CALLBACK option and I always hated setting my always changing DHCP ip address or figuring out god knows what some proxy thought I my ip was.

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.

None yet

2 participants