[PR #4987] [MERGED] Addition of Audit Log API Endpoint #6436

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

📋 Pull Request Information

Original PR: https://github.com/BookStackApp/BookStack/pull/4987
Author: @ssddanbrown
Created: 5/4/2024
Status: Merged
Merged: 5/5/2024
Merged by: @ssddanbrown

Base: developmentHead: audit_api


📝 Commits (3)

  • 3946158 API: Added audit log list endpoint
  • 67df127 API: Added to, and updated, testing to cover audit log additions
  • d54c7b4 Audit Log: Fixed bad reference to linked entity item

📊 Changes

21 files changed (+250 additions, -51 deletions)

View changed files

📝 app/Activity/ActivityQueries.php (+6 -6)
app/Activity/Controllers/AuditLogApiController.php (+28 -0)
📝 app/Activity/Controllers/AuditLogController.php (+1 -1)
📝 app/Activity/Models/Activity.php (+12 -14)
📝 app/Activity/Tools/ActivityLogger.php (+5 -5)
📝 app/Console/Commands/ClearActivityCommand.php (+1 -1)
📝 app/Entities/Models/Entity.php (+1 -1)
database/migrations/2024_05_04_154409_rename_activity_relation_columns.php (+30 -0)
dev/api/responses/audit-log-list.json (+80 -0)
📝 resources/views/common/activity-item.blade.php (+4 -4)
📝 resources/views/settings/audit.blade.php (+2 -2)
📝 routes/api.php (+3 -0)
tests/Activity/AuditLogApiTest.php (+60 -0)
📝 tests/Activity/AuditLogTest.php (+4 -4)
📝 tests/Activity/WebhookCallTest.php (+1 -1)
📝 tests/Activity/WebhookFormatTesting.php (+1 -1)
📝 tests/Activity/WebhookManagementTest.php (+1 -1)
📝 tests/Commands/ClearActivityCommandTest.php (+1 -1)
📝 tests/Settings/RecycleBinTest.php (+6 -6)
📝 tests/TestCase.php (+2 -2)

...and 1 more files

📄 Description

For #4316

Todo

  • Cover with testing
  • Review over PR changes

🔄 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/4987 **Author:** [@ssddanbrown](https://github.com/ssddanbrown) **Created:** 5/4/2024 **Status:** ✅ Merged **Merged:** 5/5/2024 **Merged by:** [@ssddanbrown](https://github.com/ssddanbrown) **Base:** `development` ← **Head:** `audit_api` --- ### 📝 Commits (3) - [`3946158`](https://github.com/BookStackApp/BookStack/commit/3946158e8821308f89ee12811f053527eedc23c2) API: Added audit log list endpoint - [`67df127`](https://github.com/BookStackApp/BookStack/commit/67df127c2618d1d2bb9bc250e86ad0753f278769) API: Added to, and updated, testing to cover audit log additions - [`d54c7b4`](https://github.com/BookStackApp/BookStack/commit/d54c7b4783b05d8a8b4b904b280453d954698732) Audit Log: Fixed bad reference to linked entity item ### 📊 Changes **21 files changed** (+250 additions, -51 deletions) <details> <summary>View changed files</summary> 📝 `app/Activity/ActivityQueries.php` (+6 -6) ➕ `app/Activity/Controllers/AuditLogApiController.php` (+28 -0) 📝 `app/Activity/Controllers/AuditLogController.php` (+1 -1) 📝 `app/Activity/Models/Activity.php` (+12 -14) 📝 `app/Activity/Tools/ActivityLogger.php` (+5 -5) 📝 `app/Console/Commands/ClearActivityCommand.php` (+1 -1) 📝 `app/Entities/Models/Entity.php` (+1 -1) ➕ `database/migrations/2024_05_04_154409_rename_activity_relation_columns.php` (+30 -0) ➕ `dev/api/responses/audit-log-list.json` (+80 -0) 📝 `resources/views/common/activity-item.blade.php` (+4 -4) 📝 `resources/views/settings/audit.blade.php` (+2 -2) 📝 `routes/api.php` (+3 -0) ➕ `tests/Activity/AuditLogApiTest.php` (+60 -0) 📝 `tests/Activity/AuditLogTest.php` (+4 -4) 📝 `tests/Activity/WebhookCallTest.php` (+1 -1) 📝 `tests/Activity/WebhookFormatTesting.php` (+1 -1) 📝 `tests/Activity/WebhookManagementTest.php` (+1 -1) 📝 `tests/Commands/ClearActivityCommandTest.php` (+1 -1) 📝 `tests/Settings/RecycleBinTest.php` (+6 -6) 📝 `tests/TestCase.php` (+2 -2) _...and 1 more files_ </details> ### 📄 Description For #4316 ## Todo - [x] Cover with testing - [x] Review over PR changes --- <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:32:15 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/BookStack#6436