Revamp the OAuth2 application admin form
Review Request #8987 — Created June 2, 2017 and submitted
The OAuth2 application form now makes use of some new widgets to make
the user experience better. The logic for the form validation has also
been revamped to take into account when validation of the redirect URIs
field fails (since they will no longer be present in the form's cleaned
data).
Manually tested the form.
Ran unit tests.
| Description | From | Last Updated |
|---|---|---|
|
Should be "Client ID" |
 chipx86
|
|
|
Not very helpful. It's just a repeat of the label. Can you update this to say exactly what this is? |
chipx86
|
|
|
W293 blank line contains whitespace |
 reviewbot
|
|
|
W291 trailing whitespace |
reviewbot
|
|
|
Blank line before conditional. |
 david
|
|
|
There seems to be a mismatched <? |
david
|
|
|
g before m |
david
|
|
|
E261 at least two spaces before inline comment |
reviewbot
|
|
|
Swap these. |
chipx86
|
|
|
Blank line between these. |
chipx86
|
|
|
This can be one statement now. |
chipx86
|
|
|
I imagine we're doing this because the parent model does clean and we don't want that one to run? It's … |
chipx86
|
|
|
clean -> clean() |
chipx86
|
|
|
Swap these. |
chipx86
|
- Change Summary:
-
Add unit tests.
- Testing Done:
-
~ Manually tested the form.
~ ~ Manually tested the form.
~ Ran unit tests. - TODO: unit tests.
Checks run (2 succeeded)
- Change Summary:
-
Mark all field labels for translation.
