Page MenuHomePhabricator
Differential D12126

Publish draft "done" status when submitting comments/updates/actions/inlines
ClosedPublic
Actions

Authored by epriestley on Mar 21 2015, 5:50 PM.
Tags
None
Referenced Files
F15471249: D12126.id29185.diff
Sat, Apr 5, 4:14 AM
F15451343: D12126.id29152.diff
Fri, Mar 28, 9:34 PM
F15451144: D12126.id.diff
Fri, Mar 28, 8:23 PM
F15450079: D12126.id29152.diff
Fri, Mar 28, 1:45 PM
F15447791: D12126.diff
Fri, Mar 28, 1:34 AM
F15446868: D12126.id29152.diff
Thu, Mar 27, 8:35 PM
F15445129: D12126.id.diff
Thu, Mar 27, 12:30 PM
F15436999: D12126.id29185.diff
Tue, Mar 25, 4:58 PM
View All Files
Subscribers
epriestley
yelirekim

Details

Reviewers
btrahan
Maniphest Tasks
T1460: Add "Done" link to an inline comment
Commits
Restricted Diffusion Commit
rP9f3210c883b9: Publish draft "done" status when submitting comments/updates/actions/inlines
Summary

Ref T1460. When a revision author updates/comments/etc on a revision, publish all their checkmarks.

This doesn't handle Diffusion/audits yet.

Test Plan

Screen_Shot_2015-03-21_at_10.50.11_AM.png (298×622 px, 29 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 29152.Mar 21 2015, 5:50 PM
epriestley retitled this revision from to Publish draft "done" status when submitting comments/updates/actions/inlines.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
epriestley added a task: T1460: Add "Done" link to an inline comment.
Herald added subscribers: epriestley, yelirekim. · View Herald TranscriptMar 21 2015, 5:50 PM
epriestley mentioned this in D12128: Publish "done" inline comment checkbox state in Diffusion.Mar 22 2015, 1:03 PM
btrahan accepted this revision.Mar 23 2015, 5:02 PM
btrahan edited edge metadata.
This revision is now accepted and ready to land.Mar 23 2015, 5:02 PM
Closed by commit rP9f3210c883b9: Publish draft "done" status when submitting comments/updates/actions/inlines (authored by epriestley, committed by epriestley). · Explain WhyMar 24 2015, 12:27 PM
This revision was automatically updated to reflect the committed changes.
epriestley mentioned this in rPcbb5a297d57b: Publish "done" inline comment checkbox state in Diffusion.

Revision Contents
Changeset List

PathSize
src/
applications/
differential/
editor/
61 lines
storage/
39 lines
infrastructure/
diff/
query/
20 lines
internationalization/
translation/
32 lines

Diff 29185

View Options

src/applications/differential/editor/DifferentialTransactionEditor.php

Loading...
View Options

src/applications/differential/storage/DifferentialTransaction.php

Loading...
View Options

src/infrastructure/diff/query/PhabricatorDiffInlineCommentQuery.php

Loading...
View Options

src/infrastructure/internationalization/translation/PhabricatorUSEnglishTranslation.php

Loading...
Phabricator · Built by Phacility