[PR #3042] [MERGED] Tag view #6122

Closed
opened 2026-02-05 10:25:01 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/BookStackApp/BookStack/pull/3042
Author: @ssddanbrown
Created: 11/6/2021
Status: Merged
Merged: 11/7/2021
Merged by: @ssddanbrown

Base: masterHead: tags_view


📝 Commits (3)

  • 929c831 Started build of tag view
  • f8f9e74 Added links to tag page
  • 899349c Added testing coverage for tag index

📊 Changes

19 files changed (+359 additions, -33 deletions)

View changed files

📝 app/Actions/TagRepo.php (+40 -10)
📝 app/Http/Controllers/TagController.php (+22 -0)
📝 resources/icons/info-filled.svg (+0 -1)
resources/icons/leaderboard.svg (+1 -0)
📝 resources/lang/en/common.php (+2 -0)
📝 resources/lang/en/entities.php (+10 -0)
📝 resources/sass/_blocks.scss (+30 -1)
📝 resources/sass/_tables.scss (+7 -1)
📝 resources/views/books/index.blade.php (+5 -0)
📝 resources/views/entities/tag-list.blade.php (+1 -9)
📝 resources/views/entities/tag-manager.blade.php (+9 -6)
resources/views/entities/tag.blade.php (+9 -0)
resources/views/form/request-query-inputs.blade.php (+8 -0)
📝 resources/views/shelves/index.blade.php (+6 -0)
resources/views/tags/index.blade.php (+56 -0)
resources/views/tags/parts/table-row.blade.php (+37 -0)
📝 routes/web.php (+4 -5)
📝 tests/Entity/TagTest.php (+92 -0)
📝 tests/TestResponse.php (+20 -0)

📄 Description

As per https://github.com/BookStackApp/BookStack/issues/738#issuecomment-928278273.

Todo

  • Add link from existing part of application
  • Check permissions are definitely enforced against tag listings
  • Cover with testing

Feature Preview

Pretty much any of the row-items can be clicked through to start a search. The right-most link will provide the same list view but with values shown and the list scoped to just a single tag name.

Screen Shot 2021-11-06 at 21 57 01


🔄 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/BookStackApp/BookStack/pull/3042 **Author:** [@ssddanbrown](https://github.com/ssddanbrown) **Created:** 11/6/2021 **Status:** ✅ Merged **Merged:** 11/7/2021 **Merged by:** [@ssddanbrown](https://github.com/ssddanbrown) **Base:** `master` ← **Head:** `tags_view` --- ### 📝 Commits (3) - [`929c831`](https://github.com/BookStackApp/BookStack/commit/929c8312bd40cffd7914ccc283cd576d68fb7bad) Started build of tag view - [`f8f9e74`](https://github.com/BookStackApp/BookStack/commit/f8f9e74992e7f0e3369d97b3e35f22f1a9bd678e) Added links to tag page - [`899349c`](https://github.com/BookStackApp/BookStack/commit/899349c4b497cbc22013bad60667222da1945763) Added testing coverage for tag index ### 📊 Changes **19 files changed** (+359 additions, -33 deletions) <details> <summary>View changed files</summary> 📝 `app/Actions/TagRepo.php` (+40 -10) 📝 `app/Http/Controllers/TagController.php` (+22 -0) 📝 `resources/icons/info-filled.svg` (+0 -1) ➕ `resources/icons/leaderboard.svg` (+1 -0) 📝 `resources/lang/en/common.php` (+2 -0) 📝 `resources/lang/en/entities.php` (+10 -0) 📝 `resources/sass/_blocks.scss` (+30 -1) 📝 `resources/sass/_tables.scss` (+7 -1) 📝 `resources/views/books/index.blade.php` (+5 -0) 📝 `resources/views/entities/tag-list.blade.php` (+1 -9) 📝 `resources/views/entities/tag-manager.blade.php` (+9 -6) ➕ `resources/views/entities/tag.blade.php` (+9 -0) ➕ `resources/views/form/request-query-inputs.blade.php` (+8 -0) 📝 `resources/views/shelves/index.blade.php` (+6 -0) ➕ `resources/views/tags/index.blade.php` (+56 -0) ➕ `resources/views/tags/parts/table-row.blade.php` (+37 -0) 📝 `routes/web.php` (+4 -5) 📝 `tests/Entity/TagTest.php` (+92 -0) 📝 `tests/TestResponse.php` (+20 -0) </details> ### 📄 Description As per https://github.com/BookStackApp/BookStack/issues/738#issuecomment-928278273. ### Todo - [x] Add link from existing part of application - [x] Check permissions are definitely enforced against tag listings - [x] Cover with testing ### Feature Preview Pretty much any of the row-items can be clicked through to start a search. The right-most link will provide the same list view but with values shown and the list scoped to just a single tag name. ![Screen Shot 2021-11-06 at 21 57 01](https://user-images.githubusercontent.com/8343178/140624968-a0d14f5b-4d0a-4abb-9bdf-48214359051e.png) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-05 10:25:01 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/BookStack#6122