Skip to content

feat: increase download limit for file attachments#1531

Merged
paustint merged 1 commit intomainfrom
feat/1527
Feb 7, 2026
Merged

feat: increase download limit for file attachments#1531
paustint merged 1 commit intomainfrom
feat/1527

Conversation

@paustint
Copy link
Contributor

@paustint paustint commented Feb 6, 2026

For the web application, paid users can now download up to 2000 attachments at a time. Free users have a reduced limit of 250 attachments per download.

Desktop and browser extension users do not have a limit imposed since these do not require server resources

Ref: resolves #1527

Copilot AI review requested due to automatic review settings February 6, 2026 15:29
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the attachment download selection limits in the Query UI to be plan- and platform-aware (higher for paid web users, lower for free web users, and intended to be unlimited for desktop/extension), and documents the new limits.

Changes:

  • Adjusts shared constants to set the base max download record count to 250 and introduces a paid-user max of 2,000.
  • Updates Query Results attachment download UI to use hasPaidPlan and platform detection to determine limits and user messaging.
  • Adds documentation clarifying plan-based limits and platform behavior.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File Description
libs/shared/constants/src/lib/shared-constants.ts Changes the global max record limit and introduces a paid-user max constant.
libs/features/query/src/QueryResults/QueryResultsAttachmentDownload.tsx Implements plan/platform-based record limit logic and updates disabled-state messaging.
libs/features/query/src/QueryResults/QueryResults.tsx Wires hasPaidPlan from app-state into the attachment download component.
apps/docs/docs/query/download-attachments.mdx Documents the updated limits and platform-specific behavior.

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

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 10 out of 10 changed files in this pull request and generated 4 comments.


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

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 10 out of 10 changed files in this pull request and generated no new comments.


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

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 12 out of 12 changed files in this pull request and generated 5 comments.


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

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 13 out of 13 changed files in this pull request and generated 5 comments.


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

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 13 out of 13 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

libs/features/query/src/QueryResults/QueryResultsAttachmentDownload.tsx:50

  • binaryCompatibleObjects is built from Object.keys(BinaryDownloadCompatibleObjectsSchema.enum) (mixed-case values like CombinedAttachment), but the component and QueryResults.tsx check membership using sobjectName.toLowerCase(). This means CombinedAttachment will never be considered compatible and the download UI will not appear for that object. Normalize the enum values when building the set (e.g., lower-case all entries) or stop lowercasing the lookup so the casing is consistent.
export const binaryCompatibleObjects = new Set(Object.keys(BinaryDownloadCompatibleObjectsSchema.enum));


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

For the web application, paid users can now download up to 2000 attachments at a time.
Free users have a reduced limit of 250 attachments per download.

Desktop and browser extension users do not have a limit imposed
since these do not require server resources

Increased download limit from 500MB to 1GB across all files in a single download.

Ref: resolves #1527
@paustint paustint merged commit 728ccf0 into main Feb 7, 2026
10 checks passed
@paustint paustint deleted the feat/1527 branch February 7, 2026 18:18
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.

Remove limit for attachment downloads

2 participants