Skip to content

Increase upload size for avatar image#6864

Merged
Gonals merged 1 commit into
mainfrom
joe-update-upload-max
Dec 22, 2021
Merged

Increase upload size for avatar image#6864
Gonals merged 1 commit into
mainfrom
joe-update-upload-max

Conversation

@Jag96

@Jag96 Jag96 commented Dec 21, 2021

Copy link
Copy Markdown
Contributor

Details

For iOS users, the minimum upload size for the avatar image is smaller than the size of the photo a user takes when selecting the Take a Photo option, so this PR updates the minimum upload size so users can use this functionality. Once #6301 (comment) is implemented we can come back to this to see if a smaller file size should be required.

Fixed Issues

$ #6628

Tests/QA

  1. On an iPhone, sign into an account and tap the profile image in the top right
  2. On the settings screen, tap the profile image again
  3. On the profile screen, tap the profile photo and select Upload photo and then Take photo
  4. Take a photo with the front camera, confirm that you do not get an error message that the photo is over 6MB

Tested On

  • Web
  • Mobile Web
  • Desktop
  • iOS
  • Android

Screenshots

N/A

@Jag96 Jag96 self-assigned this Dec 21, 2021
@Jag96 Jag96 requested a review from a team as a code owner December 21, 2021 21:13
@MelvinBot MelvinBot requested review from Gonals and removed request for a team December 21, 2021 21:14
@Gonals Gonals merged commit 8627cdb into main Dec 22, 2021
@Gonals Gonals deleted the joe-update-upload-max branch December 22, 2021 15:31
@OSBotify

Copy link
Copy Markdown
Contributor

✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

@OSBotify

Copy link
Copy Markdown
Contributor

🚀 Deployed to staging by @Gonals in version: 1.1.23-2 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 failure ❌
🍎 iOS 🍎 failure ❌
🕸 web 🕸 success ✅

@OSBotify

OSBotify commented Jan 4, 2022

Copy link
Copy Markdown
Contributor

🚀 Deployed to production by @francoisl in version: 1.1.24-8 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅

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.

3 participants