Page MenuHomePhabricator

Fill in missing applications and icons on PHID types
Closed, ResolvedPublic

Description

Some PHIDType classes have missing application or icon specifications. Today, this primarily means that feed stories don't show icons.

After D14323, you can find PHID types with missing applications in Config → PHID Types. PHID types which are clearly part of an application but don't show anything in the "Application" column have missing methods.

D14324 is an example of how to add a method call and test that it is correct.

These patches are straightforward. We will review and accept work here from contributors if you're looking for something easy to get started with.

By the end of this, we'll probably hit 100% coverage and then make the base method abstract to force implementation and prevent this from happening again in the future.

Revisions and Commits

Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision

Event Timeline

epriestley raised the priority of this task from to Normal.
epriestley updated the task description. (Show Details)
epriestley added a project: Applications.
epriestley added a subscriber: epriestley.
epriestley added a revision: Restricted Differential Revision.Oct 24 2015, 1:13 PM
epriestley added a revision: Restricted Differential Revision.Oct 24 2015, 1:16 PM
epriestley added a commit: Restricted Diffusion Commit.Oct 24 2015, 3:16 PM
epriestley added a commit: Restricted Diffusion Commit.Oct 24 2015, 4:41 PM
revi added a revision: Restricted Differential Revision.Oct 25 2015, 12:43 AM
epriestley added a commit: Restricted Diffusion Commit.Oct 25 2015, 1:12 AM
epriestley added a revision: Restricted Differential Revision.Nov 2 2015, 1:03 PM
joshuaspence closed this task as Resolved by committing Restricted Diffusion Commit.Nov 2 2015, 7:47 PM
joshuaspence added a commit: Restricted Diffusion Commit.