Remove SprintTaskStoryPointsField
ClosedPublic

Authored by JeanFred on Nov 28 2017, 2:40 PM.

Details

Reviewers
mmodell
Paladox
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Commits
rPHSP9da8f6c3f272: Remove SprintTaskStoryPointsField
Patch without arc
git checkout -b D901 && curl -L https://phabricator.wikimedia.org/D901?download=true | git apply
Summary

Since 1ddd508, the Sprint extension depends on the
global maniphest.points, and the custom field is
not needed anymore.

However they can still be displayed, which is very confusing.

This drops the the custom field and the associated tests.

Test Plan

N/A

Diff Detail

Repository
rPHSP phabricator-Sprint
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
JeanFred created this revision.Nov 28 2017, 2:40 PM
Restricted Application added a reviewer: Restricted Owners Package.Nov 28 2017, 2:40 PM
JeanFred requested review of this revision.Nov 28 2017, 2:40 PM
JeanFred updated this revision to Diff 2365.

Re-running arc unit after composer install

mmodell accepted this revision.Nov 29 2017, 11:34 AM
This revision is now accepted and ready to land.Nov 29 2017, 11:34 AM
This revision was automatically updated to reflect the committed changes.

@JeanFred: I have landed this patch for you but in the future you should be able to run arc land yourself as I have given you push access to the rPHSP phabricator-Sprint repository.

In D901#17942, @mmodell wrote:

@JeanFred: I have landed this patch for you but in the future you should be able to run arc land yourself as I have given you push access to the rPHSP phabricator-Sprint repository.

Oh, cheers @mmodell ! Appreciate it :)