GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | |||||||||||||||||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | |||||||||||||||||||
GeezerVison is showing github.com — fix(knowledge): pluralize the module header and root breadcrumb by j15z · Pull Request #6516 · simstudioai/sim Conversation
Contributor
Summary
Type of Change
TestingTested manually Checklist
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactions
All reactions
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page.
|
There was an error while loading. Please reload this page.
PR SummaryLow Risk Overview The list page metadata title, Resource.Header title, and loading skeleton title use the plural form. KB detail and document views update the root breadcrumb (and editor/loading breadcrumb variants) the same way. Singular strings for individual entities—modals, delete flows, name fallbacks like Reviewed by Cursor Bugbot for commit 4e12dcc. Bugbot is set up for automated code reviews on this repo. Configure here. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Greptile SummaryThe PR consistently renames the Knowledge Base module-level copy to “Knowledge bases” while preserving singular copy for individual entities.
Confidence Score: 5/5The PR appears safe to merge because the changes are limited to consistent module-level copy updates and preserve singular entity wording. The changed strings only affect displayed collection labels and metadata; navigation, data flow, entity fallbacks, and runtime contracts remain unchanged.
|
| Filename | Overview |
|---|---|
| apps/sim/app/workspace/[workspaceId]/knowledge/[id]/[documentId]/document.tsx | Updates every document-view root breadcrumb state to the plural module label without changing entity labels or navigation behavior. |
| apps/sim/app/workspace/[workspaceId]/knowledge/[id]/[documentId]/loading.tsx | Aligns the document loading breadcrumb with the rendered document view. |
| apps/sim/app/workspace/[workspaceId]/knowledge/[id]/base.tsx | Pluralizes the detail view’s module header and root breadcrumb while retaining singular knowledge-base name fallbacks. |
| apps/sim/app/workspace/[workspaceId]/knowledge/[id]/loading.tsx | Aligns the detail loading breadcrumb with the completed detail view. |
| apps/sim/app/workspace/[workspaceId]/knowledge/knowledge.tsx | Changes the shared collection label used by the list header, root breadcrumb, and move destination. |
| apps/sim/app/workspace/[workspaceId]/knowledge/loading.tsx | Updates the collection loading header to match the final module header. |
| apps/sim/app/workspace/[workspaceId]/knowledge/page.tsx | Updates the collection page’s browser metadata title to the plural module name. |
Reviews (1): Last reviewed commit: "fix(knowledge): pluralize the module hea..." | Re-trigger Greptile
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
4e12dcc1
Deployed
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.