flake8
-
reviewboard/oauth/forms.py (Diff revision 1) -
-
-
-
-
-
-
-
-
-
-
Review Request #9071 — Created July 11, 2017 and submitted
The Application forms can now regenerate the client secret view a new
OAuthSecretInputWidget
, which makes an API call to regenerate the
client secret of the associated widget. Doing so will clear the flag
that marks applications as "disbled for security" and they can be
subsequently enabled.Additionally, such disabled apps now have a warning present on the
application management screen. Regular disabled apps are displayed with
an italic font.
Description | From | Last Updated |
---|---|---|
W293 blank line contains whitespace |
reviewbot | |
W293 blank line contains whitespace |
reviewbot | |
W293 blank line contains whitespace |
reviewbot | |
W293 blank line contains whitespace |
reviewbot | |
W293 blank line contains whitespace |
reviewbot | |
W293 blank line contains whitespace |
reviewbot | |
E501 line too long (107 > 79 characters) |
reviewbot | |
W293 blank line contains whitespace |
reviewbot | |
W293 blank line contains whitespace |
reviewbot | |
W293 blank line contains whitespace |
reviewbot | |
W293 blank line contains whitespace |
reviewbot | |
W293 blank line contains whitespace |
reviewbot | |
Col: 7 'clientSecretText' is defined but never used. |
reviewbot | |
Col: 136 Missing semicolon. |
reviewbot | |
Col: 135 Expected an assignment or function call and instead saw an expression. |
reviewbot | |
Col: 11 Missing semicolon. |
reviewbot | |
W293 blank line contains whitespace |
reviewbot | |
W293 blank line contains whitespace |
reviewbot | |
W293 blank line contains whitespace |
reviewbot | |
W293 blank line contains whitespace |
reviewbot | |
W293 blank line contains whitespace |
reviewbot | |
W293 blank line contains whitespace |
reviewbot | |
E501 line too long (107 > 79 characters) |
reviewbot | |
W293 blank line contains whitespace |
reviewbot | |
W293 blank line contains whitespace |
reviewbot | |
W293 blank line contains whitespace |
reviewbot | |
W293 blank line contains whitespace |
reviewbot | |
W293 blank line contains whitespace |
reviewbot | |
Col: 7 'clientSecretText' is defined but never used. |
reviewbot | |
Col: 135 Expected an assignment or function call and instead saw an expression. |
reviewbot | |
Col: 136 Missing semicolon. |
reviewbot | |
Col: 11 Missing semicolon. |
reviewbot | |
E501 line too long (107 > 79 characters) |
reviewbot | |
E501 line too long (107 > 79 characters) |
reviewbot | |
Can't use trailing commas in ES5 code. |
david |
Added unit test
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 2 (+366 -9) |
jshint, flake8
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 3 (+365 -9) |
Refactor out the render() stuff to use the new
Djblets.Config.ListItemView.prototype.getRenderContext
reviewboard/templates/oauth/secret_input_widget.html (Diff revision 4) |
---|
Can't use trailing commas in ES5 code.