Switch a lot of conditional behavior over from settings.DEBUG to PRODUCTION.

Review Request #13499 — Created Jan. 10, 2024 and submitted — Latest diff uploaded

Information

Djblets
release-5.x

Reviewers

We have a lot of stuff that needs to behave differently in a development
environment than in production. We initially conditioned all of these on
the DEBUG setting, but a lot of users tend to turn that on when they
encounter problems, and then sometimes forget that they did so.

On the Review Board side, we've moved these things behind a new settings
key, PRODUCTION. This allows people to set DEBUG = True and get more
thorough logging, while not completely breaking their system. This
change brings that same key to Djblets.

  • Ran unit tests.
  • Smoke tested extension template loading.

Changes between revision 1 and 2

orig
1
2

Commits

Summary ID Author
Switch a lot of conditional behavior over from settings.DEBUG to PRODUCTION.
We have a lot of stuff that needs to behave differently in a development environment than in production. We initially conditioned all of these on the `DEBUG` setting, but a lot of users tend to turn that on when they encounter problems, and then sometimes forget that they did so. On the Review Board side, we've moved these things behind a new settings key, `PRODUCTION`. This allows people to set `DEBUG = True` and get more thorough logging, while not completely breaking their system. This change brings that same key to Djblets. Testing Done: - Ran unit tests. - Smoke tested extension template loading.
ce33328bf362089571c4c5ddcf138222ecbfb70c David Trowbridge
Switch a lot of conditional behavior over from settings.DEBUG to PRODUCTION.
We have a lot of stuff that needs to behave differently in a development environment than in production. We initially conditioned all of these on the `DEBUG` setting, but a lot of users tend to turn that on when they encounter problems, and then sometimes forget that they did so. On the Review Board side, we've moved these things behind a new settings key, `PRODUCTION`. This allows people to set `DEBUG = True` and get more thorough logging, while not completely breaking their system. This change brings that same key to Djblets. Testing Done: - Ran unit tests. - Smoke tested extension template loading.
9b4489ffd26694e8a21610332f866513eb4ee007 David Trowbridge
djblets/extensions/manager.py
Loading...