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

	F13201614: D21493.id51156.diff
	Tue, May 14, 1:42 PM

	F13191733: D21493.diff
	Sat, May 11, 11:51 PM

	F13189054: D21493.diff
	Sat, May 11, 5:41 AM

	F13183483: D21493.diff
	Fri, May 10, 9:27 AM

	Unknown Object (File)
	Tue, May 7, 9:23 AM

	Unknown Object (File)
	Tue, May 7, 5:43 AM

	Unknown Object (File)
	Fri, May 3, 10:44 PM

	Unknown Object (File)
	Fri, May 3, 9:16 AM

View All 81 Files

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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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 51349

src/applications/herald/application/PhabricatorHeraldApplication.php

Loading...

src/applications/herald/controller/HeraldTranscriptController.php

Loading...