Page MenuHomePhabricator
Differential D13826

Add additional statuses to Ponder
ClosedPublic
Actions

Authored by chad on Aug 7 2015, 8:27 PM.
Tags
None
Referenced Files
F13245410: D13826.diff
Thu, May 23, 6:00 AM
F13232886: D13826.id33394.diff
Tue, May 21, 1:33 AM
F13218470: D13826.id33387.diff
Sat, May 18, 12:37 PM
F13197301: D13826.diff
Mon, May 13, 12:04 AM
Unknown Object (File)
Fri, May 3, 9:27 AM
Unknown Object (File)
Thu, May 2, 2:39 AM
Unknown Object (File)
Mon, Apr 29, 12:37 PM
Unknown Object (File)
Thu, Apr 25, 3:01 AM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Maniphest Tasks
T9096: Add additional statuses to Ponder
Commits
Restricted Diffusion Commit
rPd2ef273ecd0a: Add additional statuses to Ponder
Summary

Ref T9096. This is a first cut at adding additional statuses, happy to add or subtract as needed... maybe even configurable? Also, the dialog doesn't seem to fire, I'll keep debugging.

Test Plan

Close and Reopen many questions. Test applicationSearch params by seeing resolved questions, all questions.

Diff Detail

Repository
rP Phabricator
Branch
ponder-statuses
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 7532
Build 8104: [Placeholder Plan] Wait for 30 Seconds
Build 8103: arc lint + arc unit

Event Timeline

chad updated this revision to Diff 33382.Aug 7 2015, 8:27 PM
chad retitled this revision from to Add additional statuses to Ponder.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
chad added a task: T9096: Add additional statuses to Ponder.
Herald added a subscriber: Korvin. · View Herald TranscriptAug 7 2015, 8:27 PM
chad updated this revision to Diff 33383.Aug 7 2015, 9:04 PM
  • Fix dialog popping up
chad updated this revision to Diff 33387.Aug 8 2015, 4:25 AM
  • Make status dialog a little more verbose
epriestley accepted this revision.Aug 8 2015, 4:49 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Aug 8 2015, 4:49 PM
Closed by commit rPd2ef273ecd0a: Add additional statuses to Ponder (authored by chad, committed by chad). · Explain WhyAug 8 2015, 5:23 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 33383

View Options

resources/sql/autopatches/20150806.ponder.status.1.sql

Loading...
View Options

resources/sql/autopatches/20150806.ponder.status.2.sql

Loading...
View Options

resources/sql/autopatches/20150806.ponder.status.3.sql

Loading...
View Options

src/applications/ponder/application/PhabricatorPonderApplication.php

Loading...
View Options

src/applications/ponder/constants/PonderQuestionStatus.php

Loading...
View Options

src/applications/ponder/controller/PonderQuestionEditController.php

Loading...
View Options

src/applications/ponder/controller/PonderQuestionStatusController.php

Loading...
View Options

src/applications/ponder/controller/PonderQuestionViewController.php

Loading...
View Options

src/applications/ponder/query/PonderQuestionQuery.php

Loading...
View Options

src/applications/ponder/query/PonderQuestionSearchEngine.php

Loading...
View Options

src/applications/ponder/storage/PonderQuestion.php

Loading...
View Options

src/applications/ponder/storage/PonderQuestionTransaction.php

Loading...
Phabricator · Built by Phacility