Skip to content

feat: Add Bulk actions bar on knowledge base table for delete actions#1371

Open
mfortman11 wants to merge 2 commits intorelease-saas-0.1from
delete-update-saas-kt
Open

feat: Add Bulk actions bar on knowledge base table for delete actions#1371
mfortman11 wants to merge 2 commits intorelease-saas-0.1from
delete-update-saas-kt

Conversation

@mfortman11
Copy link
Copy Markdown
Contributor

Addresses: https://github.ibm.com/lakehouse/tracker/issues/69215

Mirrors the Batch Actions behavior of a table in Carbon https://carbondesignsystem.com/components/data-table/usage/

Screen.Recording.2026-04-09.at.4.11.41.PM.mov

mfortman11 and others added 2 commits April 9, 2026 14:08
* sdk saas updates

* update versions

* chore: update uv.lock files after version bump

* Require API key or headers; add auth tests

Enforce authentication at OpenRAGClient construction by raising AuthenticationError when neither api_key nor extra_headers (nor OPENRAG_API_KEY env var) are provided. Also lightweight formatting cleanups in client request/header construction. Expanded tests: updated the missing-key test and added coverage for extra_headers-only auth, env-var-only auth, and explicit api_key precedence to ensure various auth modes (e.g. IBM headers) are supported and prioritized correctly.

* tag as rc1 for testing purposes

* chore: update uv.lock files after version bump

---------

Co-authored-by: Edwin Jose <edwinjose900@gmail.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@github-actions github-actions bot added community frontend 🟨 Issues related to the UI/UX tests enhancement 🔵 New feature or request labels Apr 9, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR introduces a Carbon-like bulk actions bar for the knowledge base table selection flow, and also updates both the Python and TypeScript SDKs to support an IBM authentication mode via per-request extra headers (with accompanying version bumps).

Changes:

  • Add a KnowledgeBatchActionsBar UI and swap it in when rows are selected on the knowledge page.
  • Add SDK support for “IBM auth” by allowing authentication via extra_headers / extraHeaders (instead of only API key/env var).
  • Bump SDK versions to a 0.3.0 release candidate.

Reviewed changes

Copilot reviewed 8 out of 9 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
tests/integration/sdk/test_auth.py Expands auth construction tests (missing creds, env var, extra headers, precedence).
sdks/typescript/src/types.ts Documents new extraHeaders option on the TS client.
sdks/typescript/src/client.ts Adds extraHeaders support when building request headers.
sdks/typescript/package.json Bumps TS SDK version.
sdks/python/uv.lock Bumps Python SDK version in lockfile.
sdks/python/pyproject.toml Bumps Python SDK version.
sdks/python/openrag_sdk/client.py Adds extra_headers support and adjusts auth validation/headers.
frontend/components/knowledge-batch-actions-bar.tsx New bulk actions bar component (cancel + delete).
frontend/app/knowledge/page.tsx Animates between search bar and bulk actions bar based on selection.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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

Labels

community enhancement 🔵 New feature or request frontend 🟨 Issues related to the UI/UX tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants