Page MenuHomePhabricator

Herald - add application search for transcripts
ClosedPublic

Authored by btrahan on Feb 21 2014, 8:22 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Jan 12, 8:40 AM
Unknown Object (File)
Sun, Jan 5, 4:40 PM
Unknown Object (File)
Dec 28 2024, 1:14 AM
Unknown Object (File)
Dec 12 2024, 10:22 PM
Unknown Object (File)
Dec 10 2024, 4:28 AM
Unknown Object (File)
Dec 10 2024, 2:44 AM
Unknown Object (File)
Dec 9 2024, 5:52 PM
Unknown Object (File)
Nov 29 2024, 1:54 AM

Details

Summary

this diff also makes the "test console" appear with the main search nav *and* updates application search to use the page title as the crumb rather than just search. Fixes T4399.

Test Plan

queried for transcript ids - success! queried for TX and MX - success! saved the TX and MX query and it worked again!

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

Awesome!

src/applications/herald/controller/HeraldTranscriptListController.php
110–111

Consider overriding buildApplicationCrumbs() here to add a "Transcripts" crumb, similar to the "Build Plans" crumb here:

https://secure.phabricator.com/harbormaster/plan/

btrahan updated this revision to Unknown Object (????).Feb 21 2014, 8:50 PM

override crumbs for better UI experience