flake8
-
reviewboard/admin/custom_evolutions/oauth2_provider/move_to_migrations.py (Diff revision 1) Show all issues
Review Request #12022 — Created Jan. 29, 2022 and submitted
Information | |
---|---|
david | |
Review Board | |
master | |
Reviewers | |
reviewboard | |
Django 2+ provides a new method for definings URLs,
path()
. This is
much simpler for basic cases, and makes URL patterns more readable. In
addition, the oldurl()
method has been renamed tore_path()
, and a
warning is raised when using the old name, so switch to that too.As part of this, I changed the file attachment URLs to actually parse
UUIDs using Django's new converter for that. This required making a
change in the testcreate_file_attachment_base
method to use real
UUIDs instead of the string "test-uuid".
Ran unit tests.
Commits: |
|
|||||||||
---|---|---|---|---|---|---|---|---|---|---|
Diff: |
Revision 2 (+688 -690) |
Indentation in docs.
Commits: |
|
|||||||||
---|---|---|---|---|---|---|---|---|---|---|
Diff: |
Revision 3 (+698 -700) |
Description: |
|
|||||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Commits: |
|
|||||||||||||||||||||||||||
Diff: |
Revision 4 (+716 -716) |