Skip to content

Fix landing page popup upload limit copy (3 → 8) - #18

Merged
Nishit24113 merged 1 commit into
mainfrom
fix/landing-page-upload-limit-8
Jul 28, 2026
Merged

Fix landing page popup upload limit copy (3 → 8)#18
Nishit24113 merged 1 commit into
mainfrom
fix/landing-page-upload-limit-8

Conversation

@Nishit24113

Copy link
Copy Markdown
Collaborator

What

Updates the landing page restrictions popup in LandingPage.jsx from "limited to 3 PDF document uploads" to 8, matching the real per-user default (already shown in the logged-in sidebar, LeftNav.jsx).

Why

The popup copy was stale. It is display text only — it has no connection to quota enforcement. The actual limit is stored per-user in the Cognito attribute custom:max_files_allowed (read in MainApp.js, enforced in UploadSection.jsx and the checkOrIncrementQuota Lambda). This change does not grant or alter anyone's upload quota.

Scope

  • 1 file, 1 line, text only. No logic, no functional change.

Note (separate follow-up, not in this PR)

Some existing users still have a stored limit of 3/5 from a historical deploy issue (Jan–Feb 2026) that has since been corrected. Fixing those users is a Cognito data update, tracked separately — no code change required.

The landing page restrictions popup (LandingPage.jsx) showed a stale
'3 PDF document uploads' limit. The actual per-user default is 8 -- set
by the PostConfirmation Lambda and shown in the logged-in sidebar
(LeftNav.jsx). This is display text only; it does not affect quota
enforcement, which is driven by the Cognito custom:max_files_allowed
attribute and the checkOrIncrementQuota Lambda.

@shaashvat01 shaashvat01 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

just a text change

@Nishit24113
Nishit24113 merged commit 2dbc220 into main Jul 28, 2026
@Nishit24113
Nishit24113 deleted the fix/landing-page-upload-limit-8 branch July 28, 2026 19:31
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