Differential D21493

Separate Herald transcripts into several different views
ClosedPublic
Actions

Authored by epriestley on Nov 7 2020, 12:20 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Dec 22, 12:48 AM

	Unknown Object (File)
	Mon, Dec 16, 6:26 AM

	Unknown Object (File)
	Sat, Dec 7, 5:37 PM

	Unknown Object (File)
	Wed, Dec 4, 7:01 PM

	Unknown Object (File)
	Tue, Dec 3, 9:59 AM

	Unknown Object (File)
	Mon, Nov 25, 5:43 AM

	Unknown Object (File)
	Nov 21 2024, 7:36 AM

	Unknown Object (File)
	Nov 17 2024, 12:09 PM

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13586: In Herald transcripts, surface exceptions encountered while evaluating fields
Commits: rP5408429466b5: Separate Herald transcripts into several different views

Summary

Ref T13586. The Herald transcript page has become more and more complicated over time, and recently added "Transactions" and "Profiler" sections. Split these across separate navigation tabs to limit the maximum complexity of any single view and make it easier to navigate to particular sections, like the profiler section.

Test Plan

Viewed various transcripts, saw nice digestible sections.

Diff Detail

Repository

Branch

herald1

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 25139
Build 34692: Run Core Tests
Build 34691: arc lint + arc unit

Event Timeline

epriestley created this revision.Nov 7 2020, 12:20 AM

Harbormaster completed remote builds in B24970: Diff 51156.Nov 7 2020, 12:21 AM

epriestley requested review of this revision.Nov 7 2020, 12:21 AM

Fix regexp issue.

Harbormaster completed remote builds in B25139: Diff 51329.Feb 17 2021, 4:06 PM

This revision was not accepted when it landed; it landed in state Needs Review.Feb 19 2021, 7:16 PM

Closed by commit rP5408429466b5: Separate Herald transcripts into several different views (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rP5408429466b5: Separate Herald transcripts into several different views.

Revision Contents
Changeset List

Path

Size

src/

applications/

herald/

application/

PhabricatorHeraldApplication.php

4 lines

controller/

HeraldTranscriptController.php

219 lines

Diff 51329

src/applications/herald/application/PhabricatorHeraldApplication.php

Loading...

src/applications/herald/controller/HeraldTranscriptController.php

Loading...