Page MenuHomePhabricator
Differential D21215

Don't consider empty inlines when considering whether a revision has draft comments or not
ClosedPublic
Actions

Authored by epriestley on May 4 2020, 5:29 PM.
Tags
None
Referenced Files
F18178001: D21215.id50518.diff
Sat, Aug 16, 12:51 AM
F18112312: D21215.diff
Tue, Aug 12, 4:21 PM
F18101081: D21215.id.diff
Sat, Aug 9, 12:19 PM
F18048896: D21215.diff
Sun, Aug 3, 2:28 PM
F17954540: D21215.id50528.diff
Fri, Aug 1, 4:06 AM
F17953885: D21215.id50518.diff
Fri, Aug 1, 3:34 AM
F17912634: D21215.id.diff
Jul 29 2025, 4:34 PM
F17887590: D21215.diff
Jul 29 2025, 1:47 AM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13513: Make schema changes to inlines to prepare for new inline features
Commits
rP27b7ba814a5a: Don't consider empty inlines when considering whether a revision has draft…
Summary

Ref T13513. When computing whether a revision has draft comments or not, ignore empty inlines.

Test Plan

Added empty inlines to a revision, no longer saw a yellow "draft" bubble in the list UI.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.May 4 2020, 5:29 PM
Harbormaster completed remote builds in B24335: Diff 50518.May 4 2020, 5:30 PM
epriestley requested review of this revision.May 4 2020, 5:30 PM
This revision was not accepted when it landed; it landed in state Needs Review.May 4 2020, 8:17 PM
Closed by commit rP27b7ba814a5a: Don't consider empty inlines when considering whether a revision has draft… (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rP27b7ba814a5a: Don't consider empty inlines when considering whether a revision has draft….

Revision Contents
Changeset List

PathSize
src/
applications/
differential/
query/
11 lines

Diff 50528

View Options

src/applications/differential/query/DifferentialTransactionQuery.php

Loading...
Phabricator · Built by Phacility