• 
      

    Hide the Commits field when there are no commits present.

    Review Request #13296 — Created Oct. 1, 2023 and submitted — Latest diff uploaded

    Information

    Review Board
    release-6.x

    Reviewers

    The Commits field on the main review request box is present even if
    there's no commits data to show. We end up with a label but no content.
    This happens because we specify the field should be always present
    unless this is on the diff viewer page (which has its own commits
    representation).

    We now factor in the value itself, ensuring that we only show the field
    if we know we're going to end up showing a list of commits.

    Tested viewing review requests with and without commits data. I could
    no longer reproduce the bug where only a Commits label was shown.

    Commits

    Files