Include and make use of update timestamps for all entries.

Review Request #9195 — Created Sept. 15, 2017 and submitted — Latest diff uploaded

Information

Review Board
release-3.0.x
6664046...

Reviewers

Entries for the review request page previously only had a single
timestamp, representing the time the entry was first added, and
dictating the order of the page. The new code for handling updates to
entries on review requests checked that timestamp, and while this
appeared to work in a developer server environment (possibly a timezone
issue, which is still being looked at), it wasn't working in production.

This change introduces a new "updated" timestamp, representing when an
update was made to the entry. The existing timestamp is now an "added"
timestamp, used for order and "new entry" calculations. The "updated"
timestamp factors in things like replies to reviews and state changes
for status updates. The update checks make use of this now to determine
whether the entry's new HTML and metadata should be loaded onto the
page.

Unit tests were also added for all the new timestamp logic, and to
ensure update processing is now handled appropriately when there are
updates to existing entries.

Tested with pending -> resolved status updates. Checked the timestamps
of the updates coming in and made sure that the addedTimestamp didn't
change, but that updatedTimestamp did.

Manually checked the added/updated timestamps of other entries in
response to changes I made.

All unit tests pass.

    Loading...