Add a "Last Updated By" column to the dashboard.

Review Request #12018 — Created Jan. 28, 2022 and updated — Latest diff uploaded

Information

Review Board
master

Reviewers

These changes add a new column to the review request dashboard called "Last Updated By" which displays the user that made the last update to each review request. Similar to the existing "Last Updated" column, which shows the time that the last update was made, this column is updated whenever a review request is created, edited, receives a review, is closed, or reopened.

Changes

"Last Updated By" column

  • Added a "Last Updated By" column
  • Added information about the new column to manual
  • Added tests for the new column

ReviewRequest last activity info functions

  • ReviewRequest.get_last_activity_info() also returns the user that last updated the review request
  • Review requests cache their last activity info (last updated object, changedesc, and user) in ReviewRequest.extra_data
  • Added ReviewRequest.set_last_activity_info(timestamp, updated_object, changedesc, user)
  • Modified existing uses of ReviewRequest.get_last_activity_info() to take advantage of the user field
  • Added and modified existing tests of ReviewRequest.get_last_activity_info()

Other

  • Made the WebAPI ReviewRequestLastUpdateResource return the correct user now that ReviewRequest.get_last_activity_info() includes it
  • Added tests for the column itself as well as the new last activity-related functions.
  • Ran all backend tests and observed that all passed.
  • Performed manual testing of the functionality of the column by creating, editing, reviewing, closing, and reopening review requests and observing how the column changes.

Diff Revision 5

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

orig
1
2
3
4
5
6
7
8
9
10
11
12

Commits

First Last Summary ID Author
[WIP] Add a "Last Updated By" column to the dashboard
018e55ee518b108bc942d575ff68b1acdaebb2a4 Kyle McLean
[WIP] First try at optimization of get_last_activity_info
705951a269741bc11d2dea018f3743311ea6232f Kyle McLean
Continue work on last_activity_info caching
* Update the cache when last activity changes * Add tests for get_last_activity_info * New: update last activity info when closing a review request
f0a4ae54f5632724b76515de21cbadf3850aaafe Kyle McLean
Fix import error in review_request.py
a8a16bd362d07de0b8da9ff23fc4df731372f80a Kyle McLean
Fix incorrect argument order
a7b023a845d7abefe3b1be56668e3588e64f8590 Kyle McLean
None handling for changedesc
1bcdbc1d41a3667d119b58bb4bc75952cbecdecb Kyle McLean
Fetch users in Last Updated By column in one query
1feee9b8fb362481298237575816743d3d429014 Kyle McLean
This diff has been split across 2 pages: 1 2 >
AUTHORS
dev-requirements.txt
contrib/docker/build.py
docs/releasenotes/4.0.5.rst
docs/releasenotes/4.0.6.rst
docs/releasenotes/index.rst
reviewboard/dependencies.py
reviewboard/admin/templatetags/rbadmintags.py
reviewboard/datagrids/columns.py
reviewboard/datagrids/grids.py
reviewboard/diffviewer/filediff_creator.py
reviewboard/diffviewer/opcode_generator.py
reviewboard/diffviewer/parser.py
reviewboard/diffviewer/views.py
reviewboard/diffviewer/models/filediff.py
reviewboard/diffviewer/testing/__init__.py
reviewboard/diffviewer/testing/mixins.py
reviewboard/diffviewer/tests/test_diff_parser.py
reviewboard/diffviewer/tests/test_diffx_parser.py
reviewboard/diffviewer/tests/test_download_patch_error_bundle_view.py
This diff has been split across 2 pages: 1 2 >
Loading...