Summary

Bug Fix: NOT_LOGGED_IN causes dashboard to throw an error

Review Request #2068 — Created Jan. 26, 2011 and submitted

Information

Owner

kamlani

Repository

Djblets

Branch

extensions

Bugs

Depends On

Reviewers

Groups

reviewboard

People

Description

I suspect that NOT_LOGGED_IN got missed during a merge with master. This caused the dashboard to throw a trace. This solves the error.

Testing Done

Manual

Expand All
Collapse All

Groups:: djblets
reviewboard

Branch:: extensions

Hey - just one piece of feedback (see below).

djblets/webapi/resources.py (Diff revision 1)

Generally, in RB development, we tend to put unordered lists of elements like this into alphabetical order.  So, can you update this in this order:

DISABLE_EXTENSION_FAILED,
DOES_NOT_EXIST,
ENABLE_EXTENSION_FAILED,
NOT_LOGGED_IN,
PERMISSION_DENIED,
WebAPIError

Diff:

Revision 2 (+4 -3)

Show changes

djblets/webapi/resources.py

Ship it!

```
Thanks!  Committed in 5af40ae5
```