Skip to content

feat: add knowledge base pages#657

Merged
ferruhcihan merged 3 commits into
mainfrom
APL-1134
Oct 2, 2025
Merged

feat: add knowledge base pages#657
ferruhcihan merged 3 commits into
mainfrom
APL-1134

Conversation

@ferruhcihan
Copy link
Copy Markdown
Collaborator

@ferruhcihan ferruhcihan commented Oct 1, 2025

📌 Summary

This PR adds knowledge base pages.

Ticket: https://track.akamai.com/jira/browse/APL-1134
PRs: apl-api

🔍 Reviewer Notes

Testing:

  • Update the otomi-api app _rawValues in your cluster:
        image:
            # registry: docker.io
            # repository: linode/apl-api
            pullPolicy: Always
            tag: APL-1134
  • Update the otomi-console app _rawValues in your cluster:
        image:
            # registry: docker.io
            # repository: linode/apl-console
            pullPolicy: Always
            tag: APL-1134
  • Create a pgvector database using pgvector-cluster chart in the catalog page
  • Create an embedding model using k8s-deployment chart (example below)
fullnameOverride: llama-3
image:
  repository: nginx
  pullPolicy: IfNotPresent
  tag: latest
env: []
podAnnotations: {}
podLabels:
  modelType: embedding
  modelName: llama-3
commonLabels:
  modelType: embedding
  modelName: llama-3
  • Test and verify that the new Knowledge Base pages work as expected

Considerations

  • I have tested the changes in both light and dark mode.
  • I have considered the need for new unit tests.
  • I have tested the changes on a cluster.
  • I have included relevant documentation updates.
  • I have an approved Figma design or have reflected my changes in Figma
  • I have verified that the UI/UX is consistent in major browsers (e.g., Chrome, Firefox, Safari, Edge).
  • I have tested the changes for responsiveness in different screen resolutions.
  • I have tested expected error states and verified that the user is presented with informative error messages.
  • I have tested the feature with unusual or extreme inputs (e.g., very long strings, empty states, clicking a button multiple times quickly).

Copy link
Copy Markdown

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 adds knowledge bases functionality to the application, introducing new API endpoints and pages for managing knowledge bases within teams. The feature allows users to create, view, edit, and delete knowledge bases with data sources and embedding models.

  • Adds comprehensive knowledge base API endpoints (CRUD operations)
  • Implements knowledge base overview and create/edit pages with proper validation
  • Integrates knowledge base navigation into the main application menu

Reviewed Changes

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

Show a summary per file
File Description
src/redux/otomiApi.ts Adds AI models and knowledge base/agent API endpoints with type definitions
src/pages/knowledge-bases/overview/KnowledgeBasesOverviewPage.tsx Implements knowledge base listing page with table view
src/pages/knowledge-bases/index.tsx Exports knowledge base page components
src/pages/knowledge-bases/create-edit/create-edit-knowledgeBases.validator.ts Validation schema for knowledge base form
src/pages/knowledge-bases/create-edit/KnowledgeBasesCreateEditPage.tsx Form page for creating and editing knowledge bases
src/components/NavConfig.tsx Adds knowledge bases navigation menu item
src/App.tsx Adds routing for knowledge base pages
public/i18n/en/common.json Adds translation keys for knowledge base UI

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@ElderMatt ElderMatt self-assigned this Oct 1, 2025
Comment thread src/components/NavConfig.tsx
@linode linode deleted a comment from Copilot AI Oct 2, 2025
@ferruhcihan ferruhcihan merged commit e475ef9 into main Oct 2, 2025
6 checks passed
@ferruhcihan ferruhcihan deleted the APL-1134 branch October 2, 2025 08:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants