[PR #220] [MERGED] Changes to have more assets #215 #1560

Closed
opened 2026-03-02 11:58:11 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/karakeep-app/karakeep/pull/220
Author: @kamtschatka
Created: 6/12/2024
Status: Merged
Merged: 6/23/2024
Merged by: @MohamedBassem

Base: mainHead: more-content


📝 Commits (8)

  • d84875d Allow downloading more content from a webpage and index it #215
  • c917a4c Allow downloading more content from a webpage and index it #215
  • 11367ce generalize the assets table to not be linked in particular to links
  • 2903e85 Merge branch 'hoarder-app:main' into more-content
  • 00e3b50 Merge branch 'main' into more-content
  • fe7427f Merge branch 'main' into more-content
  • ec18ac5 fix migrations post merge
  • d1302c4 fix missing asset ids in the getBookmarks call

📊 Changes

6 files changed (+1271 additions, -52 deletions)

View changed files

📝 apps/workers/crawlerWorker.ts (+71 -29)
packages/db/drizzle/0024_premium_hammerhead.sql (+31 -0)
packages/db/drizzle/meta/0024_snapshot.json (+1060 -0)
📝 packages/db/drizzle/meta/_journal.json (+7 -0)
📝 packages/db/schema.ts (+37 -3)
📝 packages/trpc/routers/bookmarks.ts (+65 -20)

📄 Description

The first PR to remove the assetids from the bookmarks table and move them to a new table
updated the UI to use the new format


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/karakeep-app/karakeep/pull/220 **Author:** [@kamtschatka](https://github.com/kamtschatka) **Created:** 6/12/2024 **Status:** ✅ Merged **Merged:** 6/23/2024 **Merged by:** [@MohamedBassem](https://github.com/MohamedBassem) **Base:** `main` ← **Head:** `more-content` --- ### 📝 Commits (8) - [`d84875d`](https://github.com/karakeep-app/karakeep/commit/d84875d7b3f4b7ece7a8fbdbe32f4cdfce307554) Allow downloading more content from a webpage and index it #215 - [`c917a4c`](https://github.com/karakeep-app/karakeep/commit/c917a4cbec8f935b9b78121a75644369dca5d5e0) Allow downloading more content from a webpage and index it #215 - [`11367ce`](https://github.com/karakeep-app/karakeep/commit/11367cec46e62d617fc18b0ef526cbeee69663b4) generalize the assets table to not be linked in particular to links - [`2903e85`](https://github.com/karakeep-app/karakeep/commit/2903e85c3e5334aed0c9b2857d877bb5cfce7dd2) Merge branch 'hoarder-app:main' into more-content - [`00e3b50`](https://github.com/karakeep-app/karakeep/commit/00e3b50023b93f3f5e0f34874c7bdaa48d2f5c26) Merge branch 'main' into more-content - [`fe7427f`](https://github.com/karakeep-app/karakeep/commit/fe7427f35638d2fc9049131b6c55bc0e87d6928f) Merge branch 'main' into more-content - [`ec18ac5`](https://github.com/karakeep-app/karakeep/commit/ec18ac5a215bf13692d4d1a4454b703238826518) fix migrations post merge - [`d1302c4`](https://github.com/karakeep-app/karakeep/commit/d1302c4af172b374a09ccfd114f6a6ab68151d5d) fix missing asset ids in the getBookmarks call ### 📊 Changes **6 files changed** (+1271 additions, -52 deletions) <details> <summary>View changed files</summary> 📝 `apps/workers/crawlerWorker.ts` (+71 -29) ➕ `packages/db/drizzle/0024_premium_hammerhead.sql` (+31 -0) ➕ `packages/db/drizzle/meta/0024_snapshot.json` (+1060 -0) 📝 `packages/db/drizzle/meta/_journal.json` (+7 -0) 📝 `packages/db/schema.ts` (+37 -3) 📝 `packages/trpc/routers/bookmarks.ts` (+65 -20) </details> ### 📄 Description The first PR to remove the assetids from the bookmarks table and move them to a new table updated the UI to use the new format --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-02 11:58:11 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/karakeep#1560
No description provided.