Always include the Issue Summary Table on the page.

Review Request #9123 — Created Aug. 6, 2017 and submitted

Information

Review Board
release-3.0.x
ff1d2ce...

Reviewers

The Issue Summary Table was conditionally added to the review request
page only if there were one or more issues filed against the review
request. This is so we didn't unnecessarily show an empty table.
However, this also meant that we had to conditionally add the view for
this table (which we used to do but no longer do in 3.0, as of a recent
change). Not having the table in there also meant we had no ability to
dynamically update the table as new data came in for the page (which is
coming soon).

This change unconditionally adds the table to the page. It does,
however, hide it if there are no open issues, so it retains the same
behavior as it did before. Future changes will build upon this to show
the table when there are issues.

Loaded a page without an Issue Summary Table and saw that no JavaScript
errors had appeared, and that the elements for the table was in the DOM
(but were hidden).

Loaded a page with issues and saw that the table was visible.

david
  1. Ship It!
  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-3.0.x (2b815b7)
Loading...