Skip to content

Keep users in Official Library after adding sources - #108

Merged
suguanYang merged 1 commit into
stagingfrom
fix/wangbinqi/library-add-stay-state
Jun 17, 2026
Merged

Keep users in Official Library after adding sources#108
suguanYang merged 1 commit into
stagingfrom
fix/wangbinqi/library-add-stay-state

Conversation

@suguanYang

Copy link
Copy Markdown
Contributor

Summary

  • keep the Official Library panel open after adding a library document
  • expose the canonical demo source id on materialized workspace source views
  • mark already-added library documents with an Added badge and remove the duplicate Add action

Validation

  • pnpm exec vitest run src/components/official-library-panel.test.ts src/components/workspace-shell.test.ts src/components/workspace-source-workflow.test.ts src/domains/sources/view.test.ts src/app/api/demo-sources/materialize/route.test.ts src/domains/sources/route-service.test.ts --reporter=verbose
  • pnpm exec eslint src/components/official-library-panel.tsx src/components/workspace-shell.tsx src/domains/sources/view.ts src/components/official-library-panel.test.ts src/components/workspace-shell.test.ts src/components/workspace-source-workflow.test.ts src/domains/sources/view.test.ts src/app/api/demo-sources/materialize/route.test.ts src/domains/sources/route-service.test.ts
  • pnpm exec tsc --noEmit --pretty false
  • git diff --check
  • pnpm build

@vercel

vercel Bot commented Jun 17, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
knowhere-notebook-staging Ready Ready Preview, Comment Jun 17, 2026 7:26am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
knowhere-notebook Ignored Ignored Jun 17, 2026 7:26am

Request Review

@suguanYang
suguanYang merged commit a464568 into staging Jun 17, 2026
7 checks passed
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.

1 participant