Skip to content

feat: Stub ruleEffects on workspacekinds list API - #1244

Closed
christian-heusel wants to merge 1 commit into
kubeflow:notebooks-v2from
christian-heusel:feat/stub-workspacekind-rule-effects
Closed

christian-heusel wants to merge 1 commit into
kubeflow:notebooks-v2from
christian-heusel:feat/stub-workspacekind-rule-effects

Conversation

@christian-heusel

Copy link
Copy Markdown
Member

Add a hard-coded ruleEffects: {"uiHide": false} attribute to each item
of the GET /api/v1/workspacekinds list response. This is a stub for
the Compatibility Selectors feature that establishes the response schema
the frontend can code against, without implementing the WorkspaceKind
filterRules evaluation itself.

uiHide is always false and is included regardless of whether the
namespaceFilter query parameter is provided.


Closes #838
Related #1043 #1033

Add a hard-coded `ruleEffects: {"uiHide": false}` attribute to each item
of the `GET /api/v1/workspacekinds` list response. This is a stub for
the Compatibility Selectors feature that establishes the response schema
the frontend can code against, without implementing the WorkspaceKind
filterRules evaluation itself.

`uiHide` is always false and is included regardless of whether the
`namespaceFilter` query parameter is provided.

Assisted-By: Claude Opus 4.8 <noreply@anthropic.com>
Signed-off-by: Christian Heusel <christian@heusel.eu>
@github-project-automation github-project-automation Bot moved this to Needs Triage in Kubeflow Notebooks Jul 10, 2026
@google-oss-prow google-oss-prow Bot added the area/backend area - related to backend components label Jul 10, 2026
@google-oss-prow
google-oss-prow Bot requested a review from andyatmiami July 10, 2026 17:53
@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 andyatmiami 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 a review from thesuperzapper July 10, 2026 17:53
@google-oss-prow google-oss-prow Bot added area/v2 area - version - kubeflow notebooks v2 size/L labels Jul 10, 2026
@christian-heusel

Copy link
Copy Markdown
Member Author

/close
Closing this in favor of #1235, the issue this PR was based on was representing a previous design, see #838 (comment).

@google-oss-prow google-oss-prow Bot closed this Jul 10, 2026
@google-oss-prow

Copy link
Copy Markdown

@christian-heusel: Closed this PR.

Details

In response to this:

/close
Closing this in favor of #1235, the issue this PR was based on was representing a previous design, see #838 (comment).

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@github-project-automation github-project-automation Bot moved this from Needs Triage to Done in Kubeflow Notebooks Jul 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/backend area - related to backend components area/v2 area - version - kubeflow notebooks v2 size/L

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant