Split up reviews/views.py

Review Request #12684 — Created Oct. 14, 2022 and submitted — Latest diff uploaded

Information

Review Board
release-6.x

Reviewers

The main reviews/views.py file was pretty huge, and was very difficult
to navigate. This change splits it up into a bunch of smaller files.

There are a couple trivial changes (adding some asserts, defining a
name attribute on BugTracker) in order to make type checkers a
bit happier, but otherwise this is purely moving things around.

Ran unit tests.

Diff Revision 2

This is not the most recent revision of the diff. The latest diff is revision 4. See what's changed.

orig
1
2
3
4

Commits

First Last Summary ID Author
Split up reviews/views.py
The main reviews/views.py file was pretty huge, and was very difficult to navigate. This change splits it up into a bunch of smaller files. There are a couple trivial changes (adding some asserts, defining a `name` attribute on `BugTracker`) in order to make type checkers a bit happier, but otherwise this is purely moving things around. Testing Done: Ran unit tests.
8ff2c48a93239f260898c7e5ecf5982d4b3e73d6 David Trowbridge
reviewboard/hostingsvcs/bugtracker.py
reviewboard/reviews/views.py
reviewboard/reviews/views/__init__.py
reviewboard/reviews/views/attachments.py
reviewboard/reviews/views/bug_trackers.py
reviewboard/reviews/views/diff_fragments.py
reviewboard/reviews/views/diffviewer.py
reviewboard/reviews/views/download_diff.py
reviewboard/reviews/views/email.py
reviewboard/reviews/views/mixins.py
reviewboard/reviews/views/new_review_request.py
reviewboard/reviews/views/review_request_detail.py
reviewboard/reviews/views/review_request_infobox.py
reviewboard/reviews/views/review_request_updates.py
reviewboard/reviews/views/root.py
Loading...