Page MenuHomePhabricator
Differential D10637

Fix incorrect maniphest.update conduit UI
ClosedPublic
Actions

Authored by chad on Oct 6 2014, 5:02 PM.
Tags
None
Referenced Files
F19039963: D10637.id25549.diff
Wed, Nov 26, 10:32 AM
F18938324: D10637.id.diff
Nov 11 2025, 3:03 AM
F18867474: D10637.diff
Nov 4 2025, 12:34 AM
F18824459: D10637.id25545.diff
Oct 23 2025, 3:49 PM
F18783254: D10637.id25550.diff
Oct 13 2025, 5:26 AM
F18768319: D10637.id.diff
Oct 8 2025, 2:58 AM
F18766273: D10637.diff
Oct 7 2025, 3:52 PM
F18626634: D10637.id25545.diff
Sep 16 2025, 1:42 AM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
btrahan
Maniphest Tasks
T6254: Error in maniphest.update conduit form
Commits
Restricted Diffusion Commit
rP923f625130cd: Fix incorrect maniphest.update conduit UI
Summary

Fixes T6254 and renames status as string. Though maybe this should go through formatStringConstants?

Test Plan

Reload Conduit page, see new text.

Diff Detail

Repository
rP Phabricator
Branch
mani-conduit
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 2749
Build 2753: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

chad updated this revision to Diff 25545.Oct 6 2014, 5:02 PM
chad retitled this revision from to Fix incorrect maniphest.update conduit UI.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added reviewers: epriestley, btrahan.
chad added a task: T6254: Error in maniphest.update conduit form.
Herald added subscribers: epriestley, Korvin. · View Herald TranscriptOct 6 2014, 5:02 PM
Harbormaster completed remote builds in B2748: Diff 25545.Oct 6 2014, 5:03 PM
epriestley requested changes to this revision.Oct 6 2014, 5:28 PM
epriestley edited edge metadata.

The types (like "optional string") should not be pht()'d. They will be properly typechecked in some future version of the API, and the typechecker will have no idea what to do with an "el stringo optionale".

This revision now requires changes to proceed.Oct 6 2014, 5:28 PM
chad updated this revision to Diff 25549.Oct 6 2014, 6:05 PM
chad edited edge metadata.
  • remove phts
epriestley accepted this revision.Oct 6 2014, 6:06 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Oct 6 2014, 6:06 PM
Harbormaster completed remote builds in B2749: Diff 25549.Oct 6 2014, 6:06 PM
Closed by commit rP923f625130cd: Fix incorrect maniphest.update conduit UI (authored by chad, committed by chad).Oct 6 2014, 6:06 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
maniphest/
conduit/
2 lines

Diff 25549

View Options

src/applications/maniphest/conduit/ManiphestConduitAPIMethod.php

Loading...
Phabricator · Built by Phacility