Skip to content

Switch build summary backend to GraphQL#3725

Merged
williamjallen merged 1 commit into
Kitware:masterfrom
williamjallen:build-summary-graphql
May 14, 2026
Merged

Switch build summary backend to GraphQL#3725
williamjallen merged 1 commit into
Kitware:masterfrom
williamjallen:build-summary-graphql

Conversation

@williamjallen
Copy link
Copy Markdown
Collaborator

The build summary page currently loads a small portion of the data from the legacy API. This PR converts that part of the page to load the same data via GraphQL instead. This change paves the way for the build summary API endpoint to be removed, while also significantly speeding up typical page load times for the build summary page.

The build summary page currently loads a small portion of the data from the legacy API. This PR converts that part of the page to load the same data via GraphQL instead. This change paves the way for the build summary API endpoint to be removed, while also significantly speeding up typical page load times for the build summary page.
@williamjallen williamjallen force-pushed the build-summary-graphql branch from af7ec6c to b7864c8 Compare May 14, 2026 15:06
Copy link
Copy Markdown
Member

@josephsnyder josephsnyder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We seem to lose the "Update" information within the "This Build" table when switching to this branch. The row in the table is missing for Nightly builds which do have update:
Before:

Image

After:

Image

Is this an intentional change?

@williamjallen
Copy link
Copy Markdown
Collaborator Author

@josephsnyder Yes, I deliberately removed the update row because the data isn't particularly meaningful, and will require substantial work to get an equivalent representation of via GraphQL. This is intended as a temporary solution while I work on plots to replace these tables in the next release.

@josephsnyder josephsnyder added this pull request to the merge queue May 14, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks May 14, 2026
@williamjallen williamjallen added this pull request to the merge queue May 14, 2026
Merged via the queue into Kitware:master with commit c52afc8 May 14, 2026
7 checks passed
@williamjallen williamjallen deleted the build-summary-graphql branch May 14, 2026 19:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants