Expensify / App

Welcome to New Expensify: a complete re-imagination of financial collaboration, centered around chat. Help us build the next generation of Expensify by sharing feedback and contributing to the code.
https://new.expensify.com
MIT License
3.54k stars 2.89k forks source link

[$100] Android -Profile - Error Message Cut Off When Uploading Large Image to Profile #49485

Closed IuliiaHerets closed 1 month ago

IuliiaHerets commented 1 month ago

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!


Version Number: v9.0.38-0 Reproducible in staging?: Y Reproducible in production?: No, prod has another wrong behavior Email or phone of affected tester (no customers): applausetester+tw345636637997@applause.expensifail.com Issue reported by: Applause Internal Team

Action Performed:

  1. Go to Account Settings > Profile.
  2. Upload a large image (>24MB).

Expected Result:

The error message should display fully when attempting to upload a large image to the profile.

Actual Result:

The error message is cut off when a large image is uploaded to the profile.

Workaround:

Unknown

Platforms:

Screenshots/Videos

https://github.com/user-attachments/assets/fb3473a9-f313-45f8-a43f-b991d68ebbd0

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~021836847837390067999
  • Upwork Job ID: 1836847837390067999
  • Last Price Increase: 2024-09-19
Issue OwnerCurrent Issue Owner: @fedirjh
melvin-bot[bot] commented 1 month ago

Triggered auto assignment to @sonialiap (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

melvin-bot[bot] commented 1 month ago

Triggered auto assignment to @puneetlath (DeployBlockerCash), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.

github-actions[bot] commented 1 month ago

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

  1. Identify the pull request that introduced this issue and revert it.
  2. Find someone who can quickly fix the issue.
  3. Fix the issue yourself.
IuliiaHerets commented 1 month ago

Prod behavior Screenshot_20240919-220025

etCoderDysto commented 1 month ago

Edited by proposal-police: This proposal was edited at 2024-09-19 19:13:57 UTC.

Proposal

Please re-state the problem that we are trying to solve in this issue.

Android -Profile - Error Message Cut Off When Uploading Large Image to Profile

What is the root cause of that problem?

We are not adding styles.w100 to the view below, and the container of AvatarWithImagePicker doesn't take full width https://github.com/Expensify/App/blob/98dc643e14378f654e45a7ce96f35be9f407c007/src/pages/settings/Profile/ProfilePage.tsx#L133

What changes do you think we should make in order to solve the problem?

Add styles.w100

What alternative solutions did you explore? (Optional)

Result Screenshot 2024-09-19 at 10 19 06 at night
allgandalf commented 1 month ago

I guess this should be treated as a new bug, we found out about this during testing (slack convo here), we didn't act on this because the alignment of the error on small screen is not great and eventually decided to report it in our bugs channel

c.c. @rushatgabhane @dangrous

grgia commented 1 month ago

Removing deploy blocker label since this is on prod

melvin-bot[bot] commented 1 month ago

Job added to Upwork: https://www.upwork.com/jobs/~021836847837390067999

melvin-bot[bot] commented 1 month ago

Triggered auto assignment to Contributor-plus team member for initial proposal review - @fedirjh (External)

melvin-bot[bot] commented 1 month ago

Upwork job price has been updated to $100

Krishna2323 commented 1 month ago

@grgia, we are handling this issue here.

melvin-bot[bot] commented 1 month ago

@sonialiap, @fedirjh Whoops! This issue is 2 days overdue. Let's get this updated quick!

fedirjh commented 1 month ago

Let's put on hold for :

melvin-bot[bot] commented 1 month ago

@sonialiap, @fedirjh Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

fedirjh commented 1 month ago

PR that fixes this issue is ready to get merged.

melvin-bot[bot] commented 1 month ago

@sonialiap, @fedirjh Whoops! This issue is 2 days overdue. Let's get this updated quick!

fedirjh commented 1 month ago

PR is deployed to staging.

melvin-bot[bot] commented 1 month ago

@sonialiap @fedirjh this issue was created 2 weeks ago. Are we close to approving a proposal? If not, what's blocking us from getting this issue assigned? Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

sonialiap commented 1 month ago

@fedirjh am I correct in that this is being handled in https://github.com/Expensify/App/pull/49203#pullrequestreview-2333712300 which is a PR for https://github.com/Expensify/App/issues/48707? So the payment for this would also be handled there, correct?

melvin-bot[bot] commented 1 month ago

@sonialiap, @fedirjh Eep! 4 days overdue now. Issues have feelings too...

etCoderDysto commented 1 month ago

@sonialiap This issue is fixed. I think we can close this one.

image

sonialiap commented 1 month ago

Thanks 👍