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
F17307537: D13966.diff
Thu, Jun 26, 10:26 AM
Unknown Object (File)
Tue, Jun 17, 5:55 PM
Unknown Object (File)
May 25 2025, 10:52 PM
Unknown Object (File)
May 25 2025, 12:24 PM
Unknown Object (File)
May 25 2025, 6:25 AM
Unknown Object (File)
May 21 2025, 7:36 PM
Unknown Object (File)
May 18 2025, 7:59 PM
Unknown Object (File)
Apr 22 2025, 11:20 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.