Remove "Recent Commits" from repository landing page
ClosedPublic
Actions

Authored by epriestley on Jul 10 2020, 5:48 PM.

Details

Reviewers: None
Maniphest Tasks: T13552: "Close Revision" actions are executed by the Message worker, and do not execute if a previously discovered commit becomes reachable from a permanent branch
Commits: rP9afc5c628757: Remove "Recent Commits" from repository landing page

Summary

Ref T13552. Currently, the repository landing page has a panel with recent commits. This is accessible by clicking "History" and usually below the fold, so it's not clearly useful.

Since I'm consolidating this code anyway to fix an issue with the import pipeline, just get rid of this history view.

Test Plan

Viewed a repository landing page, no longer saw a history panel.

Diff Detail

Repository

rP Phabricator

Branch

publish5

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 24770
Build 34164: Run Core Tests
Build 34163: arc lint + arc unit

Event Timeline

epriestley created this revision.Jul 10 2020, 5:48 PM

Harbormaster completed remote builds in B24770: Diff 50951.Jul 10 2020, 5:49 PM

epriestley requested review of this revision.Jul 10 2020, 5:49 PM

This revision was not accepted when it landed; it landed in state Needs Review.Aug 12 2020, 3:59 PM

Closed by commit rP9afc5c628757: Remove "Recent Commits" from repository landing page (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rP9afc5c628757: Remove "Recent Commits" from repository landing page.

Revision Contents
Changeset List

Path

Size

src/

applications/

diffusion/

controller/

DiffusionRepositoryController.php

51 lines

Diff 50951

View Options

src/applications/diffusion/controller/DiffusionRepositoryController.php

Remove "Recent Commits" from repository landing pageClosedPublicActions