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
Unknown Object (File)
Sun, Dec 29, 6:32 PM
Unknown Object (File)
Fri, Dec 27, 7:08 AM
Unknown Object (File)
Sun, Dec 22, 7:29 PM
Unknown Object (File)
Fri, Dec 20, 6:40 PM
Unknown Object (File)
Sun, Dec 15, 9:38 PM
Unknown Object (File)
Thu, Dec 12, 7:32 AM
Unknown Object (File)
Wed, Dec 11, 3:40 AM
Unknown Object (File)
Wed, Dec 4, 9:57 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.