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)
Wed, Jan 29, 2:05 AM
Unknown Object (File)
Fri, Jan 24, 8:23 AM
Unknown Object (File)
Tue, Jan 21, 3:31 PM
Unknown Object (File)
Tue, Jan 21, 12:44 PM
Unknown Object (File)
Tue, Jan 21, 9:43 AM
Unknown Object (File)
Fri, Jan 3, 7:17 AM
Unknown Object (File)
Dec 29 2024, 6:32 PM
Unknown Object (File)
Dec 27 2024, 7:08 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.