Page MenuHomePhabricator
Differential D12128

Publish "done" inline comment checkbox state in Diffusion
ClosedPublic
Actions

Authored by epriestley on Mar 22 2015, 1:03 PM.
Tags
None
Referenced Files
F14063758: D12128.diff
Mon, Nov 18, 8:14 PM
F14052760: D12128.diff
Fri, Nov 15, 10:41 AM
F14044323: D12128.id29184.diff
Tue, Nov 12, 3:45 PM
F14041056: D12128.diff
Mon, Nov 11, 3:43 PM
F14023293: D12128.diff
Thu, Nov 7, 12:50 AM
F14006585: D12128.id29184.diff
Mon, Oct 28, 1:37 PM
F13984292: D12128.id29184.diff
Oct 20 2024, 12:19 PM
Unknown Object (File)
Oct 13 2024, 10:29 PM
View All Files
Subscribers
epriestley

Details

Reviewers
btrahan
Maniphest Tasks
T1460: Add "Done" link to an inline comment
Commits
Restricted Diffusion Commit
rPcbb5a297d57b: Publish "done" inline comment checkbox state in Diffusion
Summary

Ref T1460. See D12126. This is essentially the same change, but for Diffusion.

This is a bit copy/pastey. I'm going to make an effort to lift inline handling into the core before pushing this in, so hopefully that will clean things up a bit.

Test Plan

Submitted stuff in Diffusion and got checkmarks to publish.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 29154.Mar 22 2015, 1:03 PM
epriestley retitled this revision from to Publish "done" inline comment checkbox state in Diffusion.
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 a subscriber: epriestley. · View Herald TranscriptMar 22 2015, 1:03 PM
btrahan accepted this revision.Mar 23 2015, 5:02 PM
btrahan edited edge metadata.
btrahan added inline comments.
src/applications/diffusion/controller/DiffusionInlineCommentController.php
73

nice

This revision is now accepted and ready to land.Mar 23 2015, 5:02 PM
Closed by commit rPcbb5a297d57b: Publish "done" inline comment checkbox state in Diffusion (authored by epriestley, committed by epriestley). · Explain WhyMar 24 2015, 12:27 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
audit/
editor/
62 lines
storage/
41 lines
diffusion/
controller/
2 lines

Diff 29184

View Options

src/applications/audit/editor/PhabricatorAuditEditor.php

Loading...
View Options

src/applications/audit/storage/PhabricatorAuditTransaction.php

Loading...
View Options

src/applications/diffusion/controller/DiffusionInlineCommentController.php

Loading...
Phabricator · Built by Phacility