linaro-ci-dashboard

Merge lp:~pfalcon/linaro-ci-dashboard/debug_propagate_exc into lp:linaro-ci-dashboard

  1. debug_propagate_exc
  2. Merge into trunk
Proposed by Paul Sokolovsky
Status: Merged
Merged at revision: 57
Proposed branch: lp:~pfalcon/linaro-ci-dashboard/debug_propagate_exc
Merge into: lp:linaro-ci-dashboard
Diff against target: 13 lines (+3/-0)
1 file modified
dashboard/settings.py (+3/-0)
To merge this branch: bzr merge lp:~pfalcon/linaro-ci-dashboard/debug_propagate_exc
Reviewer Review Type Date Requested Status
Stevan Radaković Approve
Review via email: mp+124892@code.launchpad.net

Description of the change

Add debug setting to help debug Ajax view - set to False by default, by at least kept handy. Please consider if we want to have it True by default so far.

To post a comment you must log in.
Revision history for this message
Stevan Radaković (stevanr) wrote :

Thanks Paul.
Approve +1.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'dashboard/settings.py'
2--- dashboard/settings.py 2012-09-13 11:42:02 +0000
3+++ dashboard/settings.py 2012-09-18 11:29:19 +0000
4@@ -21,6 +21,9 @@
5
6 DEBUG = True
7 TEMPLATE_DEBUG = DEBUG
8+# Set below to True to avoid swallowing any exceptions in Django
9+# and display them on console, useful for Ajax, etc. debugging.
10+DEBUG_PROPAGATE_EXCEPTIONS = False
11
12 ROOT_PATH = os.path.dirname(os.path.abspath(__file__))
13 DEPS_INSTALL_PATH = os.path.join(ROOT_PATH, "..", "lib")

Subscribers

People subscribed via source and target branches