Skip to content

Allow users to upload images larger than 5mb - #781

Merged
affabillyty merged 3 commits into
developfrom
780/bump-image-limit-to-20mb
Oct 24, 2026
Merged

affabillyty merged 3 commits into
developfrom
780/bump-image-limit-to-20mb

Conversation

@nathanktruong

@nathanktruong nathanktruong commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

Pull Request

Change Summary

  • bumps the maxSize from 5,242,880 (5MB) to 20,971,520 (20MB)
  • fixes syntax issue preventing image uploads
  • adds support for locally testing image uploads

Change Reason

PHLASK is hosting a walk this Saturday and we want new users to be able to submit images with their resources from their phone! Since modern phones output photos with large file sizes, the limit is being bumped while the compression work is in progress.

Verification [Optional]

Demonstration of beta + fix:
https://github.com/user-attachments/assets/82692e91-4fbf-4bee-87b4-7899c4feb8a2

Related Issue: #780

@nathanktruong nathanktruong changed the title bump image limit from 5mb to 20mb Allow users to upload images larger than 5mb Oct 21, 2026
Comment thread vite.config.ts Outdated
// server to the production backend
server: {
proxy: {
'/submit-image': {

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.

same for this

const submitUrl = `https://phlask.me/submit-image?type=${imageType}`;

// Allows local dev testing
const baseUrl = import.meta.env.DEV ?

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.

Do we need this?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

For testing in local development, yes, but I'll omit these from the PR to handle in a separate ticket

@RNR1 RNR1 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.

LGTM - this might need some cleanup

@nathanktruong

Copy link
Copy Markdown
Contributor Author

@RNR1 Thanks! Took out those changes, this is only the size limit now. Ready to merge.

@affabillyty
affabillyty merged commit f6e78e2 into develop Oct 24, 2026
4 checks passed

This branch was successfully deployed

1 active deployment
test — 4a88d0b6 Deployed Oct 22, 2026 by nathanktruong via build / build #2435
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.

4 participants