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

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

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.
Summary ID
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
Description From Last Updated

line too long (80 > 79 characters) Column: 80 Error code: E501

reviewbotreviewbot
Checks run (1 failed, 1 succeeded)
flake8 failed.
JSHint passed.

flake8

david
maubin
  1. Ship It!
  2. 
      
david
Review request changed
Status:
Completed
Change Summary:
Pushed to release-5.x (b1663a2)