Add the URL Avatar Service as a default service
Review Request #8482 — Created Oct. 21, 2016 and submitted
The
URLAvatarService
is a new avatar service from Djblets to allow
absolute URLs for avatars. Users do not have access to this resource.
Instead, it is intended to be used with users that are created by
extensions so that they can provide their own avatars.
- Ran unit tests.
- Saw that the new avatar service was enabled by default.
- Could not select the new avatar service in the config form.
-
-
-
-
reviewboard/avatars/tests.py (Diff revision 1) Can we reformat this?
self.assertSetEqual( set(registry), { FileUploadService, GravatarService, URLAvatarService, })
-
-
reviewboard/avatars/tests.py (Diff revision 1) Can we format this as:
[ GravatarService.avatar_service_id, ... ]
-
-
Tool: Pyflakes Processed Files: reviewboard/avatars/tests.py reviewboard/avatars/registry.py Tool: PEP8 Style Checker Processed Files: reviewboard/avatars/tests.py reviewboard/avatars/registry.py
-
-
reviewboard/avatars/tests.py (Diff revision 2) We don't really need to use
assertSetEqual
orassertListEqual
or things like that if we're passing in just sets or lists.assertEqual
will use the right behavior.Not really worth changing all these (unless you want to), but worth mentioning.