[PR #1871] [MERGED] feat: add sequential document view logs #1900

Closed
opened 2026-02-26 20:31:34 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/documenso/documenso/pull/1871
Author: @dguyen
Created: 6/29/2025
Status: Merged
Merged: 6/30/2025
Merged by: @Mythie

Base: mainHead: feat/add-sequential-document-view-logs


📝 Commits (2)

  • b40f616 feat: add sequential document view logs
  • 9a586a6 fix: remove redundant files

📊 Changes

6 files changed (+47 additions, -454 deletions)

View changed files

apps/remix/app/components/general/document/document-history-sheet-changes.tsx (+0 -26)
apps/remix/app/components/general/document/document-history-sheet.tsx (+0 -410)
📝 apps/remix/app/routes/_authenticated+/t.$teamUrl+/documents.$id._index.tsx (+1 -17)
📝 packages/lib/server-only/document/viewed-document.ts (+24 -1)
📝 packages/lib/types/document-audit-logs.ts (+18 -0)
📝 packages/lib/utils/document-audit-logs.ts (+4 -0)

📄 Description

Description

Add a new document audit log to detect when the document is viewed. This should only be visible in the document audit log page

Notes:

  1. I wanted to reuse the DOCUMENT_OPENED event and add an additional paramter to track sequential views, but it's not query-able
  2. This will log "DOCUMENT_VIEWED" before "DOCUMENT_OPENED" but i don't think it matters

🔄 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/documenso/documenso/pull/1871 **Author:** [@dguyen](https://github.com/dguyen) **Created:** 6/29/2025 **Status:** ✅ Merged **Merged:** 6/30/2025 **Merged by:** [@Mythie](https://github.com/Mythie) **Base:** `main` ← **Head:** `feat/add-sequential-document-view-logs` --- ### 📝 Commits (2) - [`b40f616`](https://github.com/documenso/documenso/commit/b40f6165dad70247fcd3c2da573da0693ac34daf) feat: add sequential document view logs - [`9a586a6`](https://github.com/documenso/documenso/commit/9a586a6807b209cdc7d32f6c9e0c9b3d187fc992) fix: remove redundant files ### 📊 Changes **6 files changed** (+47 additions, -454 deletions) <details> <summary>View changed files</summary> ➖ `apps/remix/app/components/general/document/document-history-sheet-changes.tsx` (+0 -26) ➖ `apps/remix/app/components/general/document/document-history-sheet.tsx` (+0 -410) 📝 `apps/remix/app/routes/_authenticated+/t.$teamUrl+/documents.$id._index.tsx` (+1 -17) 📝 `packages/lib/server-only/document/viewed-document.ts` (+24 -1) 📝 `packages/lib/types/document-audit-logs.ts` (+18 -0) 📝 `packages/lib/utils/document-audit-logs.ts` (+4 -0) </details> ### 📄 Description ## Description Add a new document audit log to detect when the document is viewed. This should only be visible in the document audit log page Notes: 1. I wanted to reuse the `DOCUMENT_OPENED` event and add an additional paramter to track sequential views, but it's not query-able 2. This will log "DOCUMENT_VIEWED" before "DOCUMENT_OPENED" but i don't think it matters --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-26 20:31:34 +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/documenso#1900
No description provided.