HomePhabricator

In Differential, render status for any active Drydock repository operation, not…

Description

In Differential, render status for any active Drydock repository operation, not just "Land" operations

Summary:
See PHI18. Third parties can currently define other types of Drydock operations (like "Merge Check" or "Cherry-Pick") but we won't show them in the UI.

This is a simple change which improves third-party support for now. These kinds of operations generally make sense in the upstream, but the pathways to support are longer.

Test Plan:

  • Verified that there are no other types of repository operation which we'd want to exclude in the upstream today by reviewing the "Repository Operation" subclasses.
  • Will click some buttons in production to make sure this works.

Reviewers: chad

Reviewed By: chad

Differential Revision: https://secure.phabricator.com/D18276

Details

Provenance
epriestleyAuthored on Jul 25 2017, 1:50 PM
epriestleyPushed on Jul 25 2017, 4:55 PM
Reviewer
chad
Differential Revision
D18276: In Differential, render status for any active Drydock repository operation, not just "Land" operations
Parents
rPc217d7619c5f: Make "A" hide or show all inline comments
Branches
Unknown
Tags
Unknown
Build Status
Buildable 17802
Build 23907: Run Core Tests