Never show the initial status updates entry as being new.
Review Request #9257 — Created Oct. 10, 2017 and submitted
The initial status updates entry on the review request page generally
will accompany the review request when first published, or will appear
shortly after as the result of a call to some external service or tool.
When first shown, it was appearing as new, with that state changing
shortly after when a status update changes. Showing this entry as new
doesn't really benefit anyone, and the jump in newness is a bit weird.
It also just wasn't useful as the newness never represented things like
new reviews from status updates.
This change forces the entry to never appear as new. Replises will
continue to appear as new as needed.
Unit tests pass.
Tested that the initial status update entry didn't appear new on
review request creation.
As I mentioned in a comment on the test, the superclass has some required args. I think you should probably include those in the method signature here instead of just catching it with
The superclass implementation requires
userarguments. I don't think this test is particularly valuable as-written--if the new
is_entry_newfunction didn't exist, this would crash.
InitialStatusUpdates.is_entry_new's arguments to include all required arguments.
Revision 2 (+36)
Checks run (2 succeeded)