Don't include draft diffsets in the initial latest timestamp.

Review Request #14009 — Created July 2, 2024 and submitted — Latest diff uploaded

Information

Review Board
release-7.x

Reviewers

When we load the diff viewer, we include the latest activity timestamp
in the page data. This is then used with subsequent accesses to the
last-update resource to determine whether to show the updates bubble.

The timestamp of a draft diffset was being included in the initial
activity timestamp, while it is not included in the timestamp returned
by the API. This meant that the updates bubble would pop up saying the
diff was new because the timestamp did not match.

Loaded the diff viewer with a draft diff (while viewing as an admin
user). Saw that the updates bubble no longer popped up incorrectly.

Commits

Files