Skip to content

feat(gui): add full computing unit tab#4331

Open
gracecluvohio wants to merge 8 commits intoapache:mainfrom
gracecluvohio:add-computing-unit-tab-full
Open

feat(gui): add full computing unit tab#4331
gracecluvohio wants to merge 8 commits intoapache:mainfrom
gracecluvohio:add-computing-unit-tab-full

Conversation

@gracecluvohio
Copy link
Copy Markdown
Contributor

@gracecluvohio gracecluvohio commented Mar 27, 2026

What changes were proposed in this PR?

This PR adds the rest of the features to create a new computing unit tab, including creating and deleting computing units and viewing computing unit details. It is part 3 of 3 of PR #3691.

Demonstration:

Creating a new computing unit:

Screen.Recording.2026-03-29.at.2.50.27.PM.mov

Deleting a computing unit:

Screen.Recording.2026-03-29.at.2.53.56.PM.mov

Viewing computing unit details:

Untitled.mov

Any related issues, documentation, discussions?

Closes #3690

How was this PR tested?

Tested locally

Was this PR authored or co-authored using generative AI tooling?

No

@github-actions github-actions bot added the frontend Changes related to the frontend GUI label Mar 27, 2026
@chenlica chenlica requested a review from kunwp1 March 28, 2026 23:26
@chenlica
Copy link
Copy Markdown
Contributor

@gracecluvohio Let's use "Compute" instead of "Computing Units" on the navigation bar.

@chenlica
Copy link
Copy Markdown
Contributor

@kunwp1 Please review it.

@kunwp1
Copy link
Copy Markdown
Contributor

kunwp1 commented Mar 29, 2026

@gracecluvohio Since this is part 3 of the computing unit tab, can it be reviewed independently, or should I wait for the first two PRs to be merged?

@gracecluvohio
Copy link
Copy Markdown
Contributor Author

@gracecluvohio Since this is part 3 of the computing unit tab, can it be reviewed independently, or should I wait for the first two PRs to be merged?

You should probably wait for the first two PRs to be merged. Each PR adds commits on top of the others, so there will be less files to review (and less conflicts) if earlier PRs are reviewed and merged before later ones.

@kunwp1
Copy link
Copy Markdown
Contributor

kunwp1 commented Mar 29, 2026

Thanks! Then I will wait until the two PRs are merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

frontend Changes related to the frontend GUI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Enable Direct Management of Computing Units from Dashboard

3 participants