Skip to content

docs: note that log deletion does not erase uploaded files (#933) (backport to release/1.16.1) - #934

Merged
RiskeyL merged 1 commit into
release/1.16.1from
backport/release/1.16.1/pr-933
Aug 7, 2026
Merged

docs: note that log deletion does not erase uploaded files (#933) (backport to release/1.16.1)#934
RiskeyL merged 1 commit into
release/1.16.1from
backport/release/1.16.1/pr-933

Conversation

@RiskeyL

@RiskeyL RiskeyL commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to release/1.16.1:

* docs: note that log deletion does not erase uploaded files

Adds one sentence to the Log Retention privacy Warning in both audience copies, en/zh/ja. Code-verified: neither conversation deletion nor scheduled log cleanup removes upload_files records or storage objects. Prompted by two enterprise inquiries about attachment lifecycle.

* docs: scope the uploaded-files note to conversation uploads

Review fix: the unscoped wording could be read as covering all uploads, but knowledge-document deletion does erase files; only files uploaded in conversations and runs persist.

(cherry picked from commit 5a316f8)
Copilot AI lite review requested due to automatic review settings August 7, 2026 06:30
@RiskeyL RiskeyL self-assigned this Aug 7, 2026
@dosubot dosubot Bot added size:S This PR changes 10-29 lines, ignoring generated files. documentation Improvements or additions to documentation labels Aug 7, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Backports PR #933 to release/1.16.1, updating the Logs “Log Retention” privacy warning to clarify that deleting conversations or cleaning up logs does not delete files that were uploaded within those conversations.

Changes:

  • Add a single clarifying sentence to the Log Retention <Warning> block in the English self-host and cloud docs.
  • Add equivalent translations of that sentence in the Chinese (self-host/cloud) docs.
  • Add equivalent translations of that sentence in the Japanese (self-host/cloud) docs.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
en/self-host/use-dify/monitor/logs.mdx Adds clarification in Log Retention warning about uploaded files not being erased by log/conversation deletion.
en/cloud/use-dify/monitor/logs.mdx Same clarification for the cloud audience.
zh/self-host/use-dify/monitor/logs.mdx Chinese translation of the same clarification for self-host docs.
zh/cloud/use-dify/monitor/logs.mdx Chinese translation of the same clarification for cloud docs.
ja/self-host/use-dify/monitor/logs.mdx Japanese translation of the same clarification for self-host docs.
ja/cloud/use-dify/monitor/logs.mdx Japanese translation of the same clarification for cloud docs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

<Warning>
ログには完全なユーザー会話が含まれ、機密情報が含まれる場合があります。アプリが現地のデータプライバシー規制に準拠していることを確認してください。プライバシーポリシーを公開し、必要に応じてユーザーの同意を取得してください。

会話を削除したり、ログが保持期間を過ぎて削除されたりしても、その中でアップロードされたファイルは消去されません。
<Warning>
ログには完全なユーザー会話が含まれ、機密情報が含まれる場合があります。アプリが現地のデータプライバシー規制に準拠していることを確認してください。プライバシーポリシーを公開し、必要に応じてユーザーの同意を取得してください。

会話を削除したり、ログが保持期間を過ぎて削除されたりしても、その中でアップロードされたファイルは消去されません。
@RiskeyL
RiskeyL merged commit 8b46a24 into release/1.16.1 Aug 7, 2026
7 checks passed
@RiskeyL
RiskeyL deleted the backport/release/1.16.1/pr-933 branch August 7, 2026 06:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport documentation Improvements or additions to documentation size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants