[PR #1081] [MERGED] feat: add apis to handle bookmark tags #975

New issue

Closed

opened 2026-02-25 23:36:04 +03:00 by kerem · 0 comments

kerem commented

2026-02-25 23:36:04 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/go-shiori/shiori/pull/1081
Author: @fmartingr
Created: 3/13/2025
Status: ✅ Merged
Merged: 3/15/2025
Merged by: @fmartingr

Base: master ← Head: feat/bookmark-tags-api

📝 Commits (10+)

874e27b feat: add tag search api
03372a1 feat: add apis to add/remove bookmark tags
d488337 chore: removed debug logger
58cc91a docs: updated swagger
02554f7 test: added tests
745d69e test: invalid ids
ae1ce6a feat: webapp v2
9dd6909 chore: updated swagger
6ceb77f fix: route params missing
265003c feat: added cors middleware

📊 Changes

128 files changed (+14249 additions, -465 deletions)

View changed files

➕ .cursorrules (+7 -0)
📝 .gitignore (+1 -1)
📝 Makefile (+6 -1)
📝 docs/swagger/docs.go (+130 -0)
📝 docs/swagger/swagger.json (+130 -0)
📝 docs/swagger/swagger.yaml (+85 -0)
📝 internal/cmd/server.go (+5 -0)
📝 internal/config/config.go (+6 -0)
📝 internal/database/database_tags.go (+130 -3)
📝 internal/database/database_tags_test.go (+391 -3)
📝 internal/database/database_test.go (+16 -11)
➕ internal/domains/bookmark_tags_test.go (+127 -0)
📝 internal/domains/bookmarks.go (+53 -0)
📝 internal/domains/tags.go (+5 -0)
📝 internal/http/handlers/api/v1/accounts.go (+14 -14)
📝 internal/http/handlers/api/v1/accounts_test.go (+43 -42)
📝 internal/http/handlers/api/v1/auth.go (+19 -13)
📝 internal/http/handlers/api/v1/auth_test.go (+48 -15)
➕ internal/http/handlers/api/v1/bookmark_tags_test.go (+728 -0)
📝 internal/http/handlers/api/v1/bookmarks.go (+184 -17)

...and 80 more files

📄 Description

Added new APIs to handle bookmark-tags associations
Added a way to search for tags in GET /api/v1/tags/?search=...
🚨 Breaking change: The API output now does not have a {"message":...} root key by default. API outputs are the same as exposed in the API documentation.
Experimental: new incomplete experimental web app toggable with the SHIORI_HTTP_SERVE_WEB_UI_V2 environment variable.

#658

_{🔄 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/go-shiori/shiori/pull/1081 **Author:** [@fmartingr](https://github.com/fmartingr) **Created:** 3/13/2025 **Status:** ✅ Merged **Merged:** 3/15/2025 **Merged by:** [@fmartingr](https://github.com/fmartingr) **Base:** `master` ← **Head:** `feat/bookmark-tags-api` --- ### 📝 Commits (10+) - [`874e27b`](https://github.com/go-shiori/shiori/commit/874e27bb7ec2eb9ffbbf3acbc44ff036304f0c6a) feat: add tag search api - [`03372a1`](https://github.com/go-shiori/shiori/commit/03372a14abc5544c36f784240d4d71c0a2f47eea) feat: add apis to add/remove bookmark tags - [`d488337`](https://github.com/go-shiori/shiori/commit/d488337ce80706c3f0b8857c615205ec32dd7dbf) chore: removed debug logger - [`58cc91a`](https://github.com/go-shiori/shiori/commit/58cc91a2f6de9ecd715608a4cfb4d74558a922d8) docs: updated swagger - [`02554f7`](https://github.com/go-shiori/shiori/commit/02554f7b9de05adb6405e6bb4c7c6e946dc2d7ce) test: added tests - [`745d69e`](https://github.com/go-shiori/shiori/commit/745d69e247856a08b0289dbdbaed9b80d10603e9) test: invalid ids - [`ae1ce6a`](https://github.com/go-shiori/shiori/commit/ae1ce6a3b517c22ba4b4935ee07c1970a3775029) feat: webapp v2 - [`9dd6909`](https://github.com/go-shiori/shiori/commit/9dd690989c3b52ce964057e30e1f42e4608a5b93) chore: updated swagger - [`6ceb77f`](https://github.com/go-shiori/shiori/commit/6ceb77f8ac197c17630f618a7feb67a4c204f376) fix: route params missing - [`265003c`](https://github.com/go-shiori/shiori/commit/265003cb9c3aa3db7fc1085af7b2427cf803c289) feat: added cors middleware ### 📊 Changes **128 files changed** (+14249 additions, -465 deletions) <details> <summary>View changed files</summary> ➕ `.cursorrules` (+7 -0) 📝 `.gitignore` (+1 -1) 📝 `Makefile` (+6 -1) 📝 `docs/swagger/docs.go` (+130 -0) 📝 `docs/swagger/swagger.json` (+130 -0) 📝 `docs/swagger/swagger.yaml` (+85 -0) 📝 `internal/cmd/server.go` (+5 -0) 📝 `internal/config/config.go` (+6 -0) 📝 `internal/database/database_tags.go` (+130 -3) 📝 `internal/database/database_tags_test.go` (+391 -3) 📝 `internal/database/database_test.go` (+16 -11) ➕ `internal/domains/bookmark_tags_test.go` (+127 -0) 📝 `internal/domains/bookmarks.go` (+53 -0) 📝 `internal/domains/tags.go` (+5 -0) 📝 `internal/http/handlers/api/v1/accounts.go` (+14 -14) 📝 `internal/http/handlers/api/v1/accounts_test.go` (+43 -42) 📝 `internal/http/handlers/api/v1/auth.go` (+19 -13) 📝 `internal/http/handlers/api/v1/auth_test.go` (+48 -15) ➕ `internal/http/handlers/api/v1/bookmark_tags_test.go` (+728 -0) 📝 `internal/http/handlers/api/v1/bookmarks.go` (+184 -17) _...and 80 more files_ </details> ### 📄 Description - Added new APIs to handle bookmark-tags associations - Added a way to search for tags in `GET /api/v1/tags/?search=...` - 🚨 **Breaking change**: The API output now does not have a `{"message":...}` root key by default. API outputs are the same as exposed in the API documentation. - Experimental: new incomplete experimental web app toggable with the `SHIORI_HTTP_SERVE_WEB_UI_V2` environment variable. #658 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>