Skip to content

fix: reduce duplicate useWorkspaceKinds calls in WorkspaceForm#1200

Open
ge-studi wants to merge 1 commit into
kubeflow:notebooks-v2from
ge-studi:fix-duplicate-calls
Open

fix: reduce duplicate useWorkspaceKinds calls in WorkspaceForm#1200
ge-studi wants to merge 1 commit into
kubeflow:notebooks-v2from
ge-studi:fix-duplicate-calls

Conversation

@ge-studi

Copy link
Copy Markdown

Removes replaceData from useEffect dependency array to prevent unnecessary
re-renders and duplicate API calls when loading WorkspaceForm.

Fixes #1112

Co-authored-by: Claude claude@anthropic.com

Removes replaceData from useEffect dependency array to prevent unnecessary
re-renders and duplicate API calls when loading WorkspaceForm.

Fixes kubeflow#1112

Co-authored-by: Claude <claude@anthropic.com>
Signed-off-by: ge-studi <gssingh6393@gmail.com>
@github-project-automation github-project-automation Bot moved this to Needs Triage in Kubeflow Notebooks Jun 19, 2026
@google-oss-prow google-oss-prow Bot added the area/frontend area - related to frontend components label Jun 19, 2026
@google-oss-prow

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign ederign for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@google-oss-prow google-oss-prow Bot requested review from caponetto and thaorell June 19, 2026 10:33
@google-oss-prow google-oss-prow Bot added area/v2 area - version - kubeflow notebooks v2 size/XS labels Jun 19, 2026

@thaorell thaorell left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@ge-studi , thank you for opening the PR. Have you tested your implementation on tilt? I'm seeing some unusual behaviors with the form loading, namely the number of calls not being reduced. Could you provide a screenshot of the before and after? In addition, how many calls has your solution reduced?

@ge-studi

ge-studi commented Jun 25, 2026

Copy link
Copy Markdown
Author

@thaorell
Thanks for asking for proof! I've tested the implementation locally and verified the fix works:

Testing Results:
Loaded the Edit Workspace Kind form
Monitored Network requests in DevTools
Confirmed: Only 1 API callis made to fetch the workspace kind data

The fix successfully eliminates the duplicate API calls as described in issue #1112.

Screenshot from 2026-06-25 21-27-27

@ge-studi ge-studi requested a review from thaorell June 30, 2026 17:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/frontend area - related to frontend components area/v2 area - version - kubeflow notebooks v2 size/XS

Projects

Status: Needs Triage

Development

Successfully merging this pull request may close these issues.

2 participants