Skip to content

fix(web): update data export download expiry copy to 24 hours - #5187

Merged
St0rmz1 merged 1 commit into
mainfrom
fix/data-export-expiry-copy
Aug 10, 2026
Merged

fix(web): update data export download expiry copy to 24 hours#5187
St0rmz1 merged 1 commit into
mainfrom
fix/data-export-expiry-copy

Conversation

@St0rmz1

@St0rmz1 St0rmz1 commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Summary

Updates the copy on the Data Exports page (/data-exports) to reflect that downloads expire 24 hours after the export is ready, instead of 7 days.

Changes

  • apps/web/src/app/(app)/data-exports/DataExportsClient.tsx: "downloads expire 7 days after that" → "downloads expire 24 hours after that"

Notes

Copy-only change; no logic affected.

Co-authored-by: kiloconnect[bot] <240665456+kiloconnect[bot]@users.noreply.github.com>
@St0rmz1
St0rmz1 enabled auto-merge (squash) August 10, 2026 19:17
The export includes your App Builder project titles and the prompt prefixes recorded
with your usage history. Large accounts can take a while to generate. We&apos;ll email
you when it&apos;s ready, and downloads expire 7 days after that.
you when it&apos;s ready, and downloads expire 24 hours after that.

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.

WARNING: New copy says downloads expire after 24 hours, but the backend still expires exports 7 days after completion

When an export completes, the user-data-export worker sets expires_at = now() + interval '7 days' (services/user-data-export/src/databases.ts:117), and the presigned download URL TTL is derived from that expires_at (services/user-data-export/src/index.ts, downloadExpiration). This PR changes only the UI copy, so after merge the page will tell users downloads expire in 24 hours while they actually remain available for 7 days. If the intent is a real 24-hour expiry, the worker change needs to ship with (or before) this copy; otherwise the copy should stay at 7 days. Note: the 24-hour figure matches the separate re-request throttle (apps/web/src/routers/user-exports-router.ts: "You can request one data export every 24 hours"), which is a different limit.


Reply with @kilocode-bot fix it to have Kilo Code address this issue.

@kilo-code-bot

kilo-code-bot Bot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: 1 Issue Found | Recommendation: Address before merge

Executive Summary

The updated copy claims downloads expire 24 hours after the export is ready, but the user-data-export backend still sets expires_at to 7 days on completion, so the new copy contradicts actual behavior.

Overview

Severity Count
CRITICAL 0
WARNING 1
SUGGESTION 0
Issue Details (click to expand)

WARNING

File Line Issue
apps/web/src/app/(app)/data-exports/DataExportsClient.tsx 153 Copy says downloads expire in 24 hours, but the backend (services/user-data-export/src/databases.ts:117) still expires exports 7 days after completion
Files Reviewed (1 file)
  • apps/web/src/app/(app)/data-exports/DataExportsClient.tsx - 1 issue

Fix these issues in Kilo Cloud


Reviewed by kimi-k3 · Input: 77.9K · Output: 8.3K · Cached: 808.8K

Review guidance: REVIEW.md from base branch main

@St0rmz1
St0rmz1 merged commit b55fb09 into main Aug 10, 2026
13 checks passed
@St0rmz1
St0rmz1 deleted the fix/data-export-expiry-copy branch August 10, 2026 19:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants