Page MenuHomePhabricator

Modernize transaction comment editor
ClosedPublic

Authored by epriestley on Jun 29 2015, 1:38 PM.
Tags
None
Referenced Files
F15441798: D13469.id32593.diff
Wed, Mar 26, 7:58 PM
F15439852: D13469.id32597.diff
Wed, Mar 26, 9:49 AM
F15438255: D13469.id.diff
Tue, Mar 25, 11:52 PM
F15431409: D13469.diff
Mon, Mar 24, 12:11 PM
F15382150: D13469.id32593.diff
Fri, Mar 14, 10:58 AM
F15336038: D13469.diff
Sat, Mar 8, 6:09 PM
Unknown Object (File)
Mon, Mar 3, 4:25 PM
Unknown Object (File)
Feb 11 2025, 9:34 PM
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.