Page MenuHomePhabricator

Allow Nuance source definitions to add actions to source views
ClosedPublic

Authored by epriestley on Aug 22 2015, 10:40 PM.
Tags
None
Referenced Files
F15456951: D13966.id33725.diff
Sun, Mar 30, 1:16 PM
F15441571: D13966.diff
Wed, Mar 26, 6:50 PM
F15433512: D13966.id.diff
Mon, Mar 24, 11:33 PM
F15425562: D13966.id33702.diff
Sun, Mar 23, 4:55 AM
F15422110: D13966.id33702.diff
Sat, Mar 22, 5:34 AM
F15419125: D13966.id33725.diff
Fri, Mar 21, 3:46 AM
F15396845: D13966.id.diff
Sun, Mar 16, 4:36 PM
F15381746: D13966.diff
Fri, Mar 14, 9:12 AM
Subscribers
None

Details

Summary

Ref T8783. If you have a source (like a "report bug" form), let it put a link (like "View Form") on the source detail page.

This also straightens out getting definitions from sources, which had a bug with the modern way we do PhutilClassMapQuery.

Specifically, if you called the old mechanism on two different sources, they'd return the same definition object, but they need to return different definitions.

Test Plan

Screen Shot 2015-08-22 at 3.38.58 PM.png (718×1 px, 124 KB)

Screen Shot 2015-08-22 at 3.39.01 PM.png (718×1 px, 81 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Allow Nuance source definitions to add actions to source views.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.

Those screenshots were in the wrong order -- the form already existed, I'm just showing where the link takes you.

chad edited edge metadata.
This revision is now accepted and ready to land.Aug 22 2015, 11:37 PM
This revision was automatically updated to reflect the committed changes.