Skip to content

[ci] react pr details page - #15748

Open
cjllanwarne wants to merge 7 commits into
hail-is:mainfrom
cjllanwarne:cjl_react_ci_pr_page
Open

cjllanwarne wants to merge 7 commits into
hail-is:mainfrom
cjllanwarne:cjl_react_ci_pr_page

Conversation

@cjllanwarne

Copy link
Copy Markdown
Collaborator

Change Description

React version of the PR details page, including:

  • auto refresh while running
  • less enormous jobs list (just show an overall state overview, plus the currently running, failed and erroring jobs)
  • ability to list failed jobs in the build history (to more easily spot patterns and genuinely broken tests)

Note: some overlap with #15736. Will need to resolve that when one or the other merges first.

Screenshot:

Using a recent PR as an example:

image

Security Assessment

Delete all except the correct answer:

  • This change potentially impacts the Hail Batch instance as deployed by Broad Institute in GCP

Impact Rating

  • This change has a low security impact

Impact Description

UI only change, on a developer-facing page

Appsec Review

  • Required: The impact has been assessed and approved by appsec

@cjllanwarne
cjllanwarne requested a review from grohli September 9, 2026 20:56
@cjllanwarne
cjllanwarne requested a review from a team as a code owner September 9, 2026 20:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant