Skip to content

env configured global bookmarks - #416

Draft
Bohreromir wants to merge 5 commits into
feat-277-bookmarks-uifrom
feat-277-global-bookmarks
Draft

Bohreromir wants to merge 5 commits into
feat-277-bookmarks-uifrom
feat-277-global-bookmarks

Conversation

@Bohreromir

@Bohreromir Bohreromir commented Oct 27, 2026 •

Copy link
Copy Markdown
Member

Description

Docs

Release note

Checklist

Author

  • AI review done, findings addressed
  • Tests added or updated
  • Changelog updated (if the repo keeps one)

Reviewer

  • Useful comments and logging
  • Docs added or updated per the style guide

Acceptance

  • Check for release notes and docs

@Bohreromir
Bohreromir added this pull request to stack #415 October 27, 2026 14:41
@github-actions

github-actions Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 65.38% 8893 / 13600
🔵 Statements 64.65% 11979 / 18527
🔵 Functions 67% 2849 / 4252
🔵 Branches 55.39% 5072 / 9156
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
src/hooks.server.ts 0% 0% 0% 0% 18-140
src/lib/components/dashboard/BookmarkCard.svelte 100% 100% 100% 100%
src/lib/components/dashboard/DashboardBookmarks.svelte 98.92% 85.71% 100% 100% 42
src/lib/components/layout/sidebar/Sidebar.svelte 0% 0% 0% 0% 116-286
src/lib/dashboard/types.ts 100% 100% 100% 100%
src/lib/server/dashboard/global-bookmarks.ts 60% 66.66% 66.66% 58.82% 48-56
src/routes/(app)/+layout.server.ts 100% 100% 100% 100%
src/routes/(app)/+layout.svelte 0% 0% 0% 0% 11-102
src/routes/(app)/+page.svelte 86.66% 83.33% 100% 87.5% 5-17
Generated in workflow #1163 for commit a627bf7 by the Vitest Coverage Report Action

@Bohreromir Bohreromir changed the title global bookmarks env configured global bookmarks Oct 27, 2026
…277)

Administrators can configure read-only bookmarks for every user as a JSON array, also via the Helm chart (features.bookmarks.global). They are shown before the user's own bookmarks on the dashboard and in the sidebar, also without a signed-in user. Invalid configuration stops the server at startup.
features.bookmarks.global must be a YAML list. A JSON string, e.g. copied from the README, was encoded again and stopped the application at startup; Helm now fails at render time with a clear message instead.
The sidebar and the dashboard both list global bookmarks first via bookmarksInDisplayOrder. The GlobalBookmark alias is gone in favour of BookmarkLink.
@Bohreromir
Bohreromir force-pushed the feat-277-global-bookmarks branch from cd1d7c2 to a627bf7 Compare October 27, 2026 15:48
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