Despecialize status handling in Maniphest Reports
ClosedPublic
Actions

Authored by epriestley on Mar 21 2014, 11:50 AM.

Tags

None

Referenced Files

	F14099937: D8580.diff
	Tue, Nov 26, 1:01 PM

	Unknown Object (File)
	Mon, Nov 18, 11:31 AM

	Unknown Object (File)
	Wed, Nov 13, 5:37 PM

	Unknown Object (File)
	Sun, Nov 10, 9:47 PM

	Unknown Object (File)
	Sun, Nov 10, 9:47 PM

	Unknown Object (File)
	Sun, Nov 10, 12:54 AM

	Unknown Object (File)
	Sat, Nov 9, 12:01 AM

	Unknown Object (File)
	Fri, Nov 8, 5:20 PM

View All Files

Subscribers

epriestley

Details

Reviewers

btrahan

Maniphest Tasks

T1812: Allow for arbitrary values for task status

Commits

Restricted Diffusion Commit
rP2a6d9304807d: Despecialize status handling in Maniphest Reports

Summary

Ref T1812. This is mega gross but Facts is too far away to do this right for now.

Test Plan

bleh gross

Looked at reports, saw same data as before.

Diff Detail

Repository

rP Phabricator

Branch

task2

Lint

Lint Passed

Severity	Location	Code	Message
Advice	src/applications/maniphest/controller/ManiphestReportController.php:658	XHP16	TODO Comment

Unit

No Test Coverage

Event Timeline

epriestley updated this revision to Diff 20357.Mar 21 2014, 11:50 AM

epriestley retitled this revision from to Despecialize status handling in Maniphest Reports.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: btrahan.

epriestley added a task: T1812: Allow for arbitrary values for task status.

Herald added a subscriber: epriestley.Mar 21 2014, 11:50 AM

btrahan accepted this revision.Mar 25 2014, 7:03 PM

btrahan edited edge metadata.

This revision is now accepted and ready to land.Mar 25 2014, 7:03 PM

Closed by commit rP2a6d9304807d (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

maniphest/

controller/

ManiphestReportController.php

32 lines

Diff 20357

View Options

src/applications/maniphest/controller/ManiphestReportController.php

Despecialize status handling in Maniphest ReportsClosedPublicActions