Skip to content

[HS-1572480] Fix CSV upload size check#1657

Open
canac wants to merge 1 commit intomainfrom
hs-1572480-csv-size
Open

[HS-1572480] Fix CSV upload size check#1657
canac wants to merge 1 commit intomainfrom
hs-1572480-csv-size

Conversation

@canac
Copy link
Contributor

@canac canac commented Mar 20, 2026

Description

Fix CSV size to check size in MB, not KB.

HS-1572480

Testing

  • Don't worry about testing - I can test in preview real quick before merging
  • Go to stage.mpdx.org/accountLists/6bbc265f-ec81-4b23-b297-5bd298c4d619/tools/import/csv?tab=upload
  • Upload a 200K CSV
  • Check that it accepts it

Checklist:

  • I have given my PR a title with the format "MPDX-(JIRA#) (summary sentence max 80 chars)"
  • I have applied the appropriate labels (Add the label "Preview" to automatically create a preview environment)
  • I have run the Claude Code /pr-review command locally and fixed any relevant suggestions
  • I have requested a review from another person on the project
  • I have tested my changes in preview or in staging
  • I have cleaned up my commit history

@canac canac requested a review from wjames111 March 20, 2026 19:07
@canac canac self-assigned this Mar 20, 2026
@canac canac added the Preview Environment Add this label to create an Amplify Preview label Mar 20, 2026
@github-actions
Copy link
Contributor

Preview branch generated at https://hs-1572480-csv-size.d3dytjb8adxkk5.amplifyapp.com

@github-actions
Copy link
Contributor

Bundle sizes [mpdx-react]

Compared against 14a1c22

No significant changes found

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Preview Environment Add this label to create an Amplify Preview

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant