bookstack/bookstack
bookstack/bookstack
Watch
44
261
Fork
You've already forked bookstack
 32
Code Issues 698 Pull requests 8 Releases 240 Activity Actions 26

Audit Log: Applied display timezone to dates #6170

Open
assyrus-favolo wants to merge 1 commit from assyrus-favolo/bookstack:fix-audit-display-timezone into development
pull from: assyrus-favolo/bookstack:fix-audit-display-timezone
merge into: bookstack:development
Conversation 0 Commits 1 Files changed 1 +1 -1
assyrus-favolo commented 2026-07-13 16:21:21 +02:00
First-time contributor
Copy link

Uses the shared date formatter for audit log activity dates so APP_DISPLAY_TIMEZONE is respected while APP_TIMEZONE can remain UTC for storage.

Related to issue #4786 and PR #5790, which introduced APP_DISPLAY_TIMEZONE and updated most in-app absolute date displays but left the audit log table rendering the raw created_at value.

Details

Checklist

  • I have read the BookStack community rules.
  • This PR does not feature significant use of LLM/AI generation as per the community rules above.

Disclaimer: yes it is all generated through an LLM, but it's a trivial change, I'm open to rework it myself if, for example, you want a less verbose commit message or to extend the scope to other places where the date conversion may have been skipped to make this commit encapsulate the whole change appropriately.

Uses the shared date formatter for audit log activity dates so APP_DISPLAY_TIMEZONE is respected while APP_TIMEZONE can remain UTC for storage. Related to issue #4786 and PR #5790, which introduced APP_DISPLAY_TIMEZONE and updated most in-app absolute date displays but left the audit log table rendering the raw created_at value. ## Details <!-- Write details of your pull request in here --> <!-- Include references to any relevant issues/discussions --> ## Checklist <!-- Put an 'x' in between the brackets below to confirm these elements --> - [x] I have read the [BookStack community rules](https://www.bookstackapp.com/about/community-rules/). - [ ] This PR does not feature significant use of LLM/AI generation as per the community rules above. Disclaimer: yes it is all generated through an LLM, but it's a trivial change, I'm open to rework it myself if, for example, you want a less verbose commit message or to extend the scope to other places where the date conversion may have been skipped to make this commit encapsulate the whole change appropriately.
Audit Log: Applied display timezone to dates
Some checks are pending
analyse-php / build (pull_request) Blocked by required conditions
Details
lint-php / build (pull_request) Blocked by required conditions
Details
test-migrations / build (8.2) (pull_request) Blocked by required conditions
Details
test-migrations / build (8.3) (pull_request) Blocked by required conditions
Details
test-migrations / build (8.4) (pull_request) Blocked by required conditions
Details
test-migrations / build (8.5) (pull_request) Blocked by required conditions
Details
test-php / build (8.2) (pull_request) Blocked by required conditions
Details
test-php / build (8.3) (pull_request) Blocked by required conditions
Details
test-php / build (8.4) (pull_request) Blocked by required conditions
Details
test-php / build (8.5) (pull_request) Blocked by required conditions
Details
f0f5b62c9d 
Uses the shared date formatter for audit log activity dates so APP_DISPLAY_TIMEZONE is respected while APP_TIMEZONE can remain UTC for storage.

Related to issue #4786 and PR #5790, which introduced APP_DISPLAY_TIMEZONE and updated most in-app absolute date displays but left the audit log table rendering the raw created_at value.
Some checks are pending
analyse-php / build (pull_request) Blocked by required conditions
Details
lint-php / build (pull_request) Blocked by required conditions
Details
test-migrations / build (8.2) (pull_request) Blocked by required conditions
Details
test-migrations / build (8.3) (pull_request) Blocked by required conditions
Details
test-migrations / build (8.4) (pull_request) Blocked by required conditions
Details
test-migrations / build (8.5) (pull_request) Blocked by required conditions
Details
test-php / build (8.2) (pull_request) Blocked by required conditions
Details
test-php / build (8.3) (pull_request) Blocked by required conditions
Details
test-php / build (8.4) (pull_request) Blocked by required conditions
Details
test-php / build (8.5) (pull_request) Blocked by required conditions
Details
This pull request can be merged automatically.
Some workflows are waiting to be reviewed.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u fix-audit-display-timezone:assyrus-favolo-fix-audit-display-timezone
git switch assyrus-favolo-fix-audit-display-timezone

Merge

Merge the changes and update on Forgejo.
git switch development
git merge --no-ff assyrus-favolo-fix-audit-display-timezone
git switch assyrus-favolo-fix-audit-display-timezone
git rebase development
git switch development
git merge --ff-only assyrus-favolo-fix-audit-display-timezone
git switch assyrus-favolo-fix-audit-display-timezone
git rebase development
git switch development
git merge --no-ff assyrus-favolo-fix-audit-display-timezone
git switch development
git merge --squash assyrus-favolo-fix-audit-display-timezone
git switch development
git merge --ff-only assyrus-favolo-fix-audit-display-timezone
git switch development
git merge assyrus-favolo-fix-audit-display-timezone
git push origin development
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
Focus: A11y
Accessibility

  
Focus: Admin/Meta

   
Focus: Authentication

   
Focus: Back-End

   
Focus: Database

   
Focus: Design & UX

   
Focus: Editor - Markdown

   
Focus: Editor - WYSIWYG

   
Focus: Export System

   
Focus: Front-End

   
Focus: Translations

   
Focus: View Customization

   
Is: Docs Update

   
Is: Enhancement

   
Is: Priority

   
Is: Security

   
Is: Upstream
  
Status
Blocked

  
Status
Open to discussion

  
Status
Out of scope

  
Status
Pending Validation

  
Type
API Request

  
Type
Bug Report

  
Type
Feature Request

  
Type
Happy feedback

  
Type
Maintenance

  
Type
Question

  
Type
Support

No labels
Focus: A11y
Focus: Admin/Meta
Focus: Authentication
Focus: Back-End
Focus: Database
Focus: Design & UX
Focus: Editor - Markdown
Focus: Editor - WYSIWYG
Focus: Export System
Focus: Front-End
Focus: Translations
Focus: View Customization
Is: Docs Update
Is: Enhancement
Is: Priority
Is: Security
Is: Upstream
Status
Blocked
Status
Open to discussion
Status
Out of scope
Status
Pending Validation
Type
API Request
Type
Bug Report
Type
Feature Request
Type
Happy feedback
Type
Maintenance
Type
Question
Type
Support
Milestone
Clear milestone
No items
No milestone
Assignees
Clear assignees
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
bookstack/bookstack!6170
No description provided.