Differential D21059

Add an "isDone" flag to "transaction.search" for Differential inline comments
ClosedPublic
Actions

Authored by epriestley on Apr 5 2020, 4:03 PM.

Details

Reviewers: None
Commits: rPff348b40f937: (stable) Add an "isDone" flag to "transaction.search" for Differential inline…
rPf1d1ec3d77f1: Add an "isDone" flag to "transaction.search" for Differential inline comments

Summary

See PHI1684. Expose the published state of the "Done" checkbox to the API.

Test Plan

Made API calls on a comment in all four states, got correct published states via the API in all cases.

Diff Detail

Repository

rP Phabricator

Branch

inline1

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 23991
Build 33018: Run Core Tests
Build 33017: arc lint + arc unit

Event Timeline

epriestley created this revision.Apr 5 2020, 4:03 PM

Harbormaster completed remote builds in B23991: Diff 50172.Apr 5 2020, 4:05 PM

epriestley requested review of this revision.Apr 5 2020, 4:05 PM

This revision was not accepted when it landed; it landed in state Needs Review.Apr 5 2020, 4:36 PM

Closed by commit rPf1d1ec3d77f1: Add an "isDone" flag to "transaction.search" for Differential inline comments (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rPf1d1ec3d77f1: Add an "isDone" flag to "transaction.search" for Differential inline comments.

epriestley added a commit: rPff348b40f937: (stable) Add an "isDone" flag to "transaction.search" for Differential inline….

Revision Contents
Changeset List

Path

Size

src/

applications/

differential/

xaction/

DifferentialRevisionInlineTransaction.php

9 lines

Diff 50172

View Options

src/applications/differential/xaction/DifferentialRevisionInlineTransaction.php

Add an "isDone" flag to "transaction.search" for Differential inline commentsClosedPublicActions