Page MenuHomePhabricator

Modernize transaction comment editor
ClosedPublic

Authored by epriestley on Jun 29 2015, 1:38 PM.
Tags
None
Referenced Files
F14048193: D13469.id.diff
Thu, Nov 14, 7:02 AM
F14046704: D13469.id32597.diff
Wed, Nov 13, 10:56 PM
F14045505: D13469.diff
Wed, Nov 13, 5:25 AM
F14045239: D13469.diff
Wed, Nov 13, 2:24 AM
F14043279: D13469.diff
Tue, Nov 12, 8:57 AM
F14004372: D13469.id32597.diff
Sat, Oct 26, 7:25 PM
F13959750: D13469.id.diff
Oct 14 2024, 8:36 PM
Unknown Object (File)
Oct 13 2024, 1:38 AM
Subscribers

Details

Reviewers
btrahan
Maniphest Tasks
T8703: Unable to edit macro comments
Commits
Restricted Diffusion Commit
rPb39f5d651c42: Modernize transaction comment editor
Summary

Fixes T8703. The URI handling here was a little unusual.

Test Plan
  • Edited and deleted comments in several applications, including Macro.
  • As an admin, deleted others' comments.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Modernize transaction comment editor.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Jun 29 2015, 6:04 PM
This revision was automatically updated to reflect the committed changes.