Differential D11192

Clarify Tasks listed on Project Profiles
ClosedPublic
Actions

Authored by chad on Jan 4 2015, 3:56 AM.

Details

Reviewers

epriestley

Maniphest Tasks

T6772: On the project page, it is not indicated that the "Open Tasks" list is incomplete

Commits

Restricted Diffusion Commit
rP3b09d420699e: Clarify Tasks listed on Project Profiles

Summary

Fixes T6772. Adds additional checks and language. Happy to tweak language.

Test Plan

Test a Project with 3, 10, and 11 tasks.

Diff Detail

Repository

rP Phabricator

Branch

open-task-count

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 3543
Build 3551: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

chad updated this revision to Diff 26860.Jan 4 2015, 3:56 AM

chad retitled this revision from to Clarify Tasks listed on Project Profiles.

chad updated this object.

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

chad added a reviewer: epriestley.

chad added a task: T6772: On the project page, it is not indicated that the "Open Tasks" list is incomplete.

Herald added subscribers: epriestley, Korvin. · View Herald TranscriptJan 4 2015, 3:56 AM

epriestley accepted this revision.Jan 4 2015, 4:32 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Jan 4 2015, 4:32 PM

Maybe "View All" is useful even if all are shown, in case you want to filter/expand/etc? You could call it "View in Maniphest" or something if you're worried about "All" being confusing.

Closed by commit rP3b09d420699e: Clarify Tasks listed on Project Profiles (authored by chad, committed by chad). · Explain WhyJan 4 2015, 5:03 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

project/

controller/

PhabricatorProjectProfileController.php

20 lines

Diff 26860

View Options

src/applications/project/controller/PhabricatorProjectProfileController.php

Clarify Tasks listed on Project ProfilesClosedPublicActions