Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[$250] [HOLD for payment 2024-06-11] Uploading a Group Chat custom avatar that is too large shows badly formatted error message #40199

Closed
marcaaron opened this issue Apr 12, 2024 · 17 comments
Assignees
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Daily KSv2 External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors

Comments

@marcaaron
Copy link
Contributor

marcaaron commented Apr 12, 2024

Coming from #39757 (comment)

Reproduction:

  1. Create a Group Chat
  2. Navigate to report details page
  3. Upload a custom avatar that is too large

Expected:

The error message displayed is taking up the normal width of the screen.

Actual:

Image

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~013621cec02e1951b1
  • Upwork Job ID: 1800948188003727972
  • Last Price Increase: 2024-06-12
Issue OwnerCurrent Issue Owner: @
@melvin-bot melvin-bot bot added the Monthly KSv2 label Apr 16, 2024
@s77rt
Copy link
Contributor

s77rt commented May 13, 2024

@Expensify/design I found this inconsistency in the avatar error message, which one is the correct one?

Screenshot 2024-05-13 at 11 36 55 AM

@shawnborton
Copy link
Contributor

Hmm I think I lean towards the right version being more correct. Thoughts @Expensify/design?

@dannymcclain
Copy link
Contributor

Yeah I think so too. I think it makes more sense with the rest of our app for it to span the full width.

@s77rt
Copy link
Contributor

s77rt commented May 17, 2024

Working on a similar issue on #42088 (comment). Checking if we can fix that issue there too

@melvin-bot melvin-bot bot added the Reviewing Has a PR in review label May 29, 2024
@s77rt
Copy link
Contributor

s77rt commented May 29, 2024

Just raised a PR

@melvin-bot melvin-bot bot added Weekly KSv2 and removed Monthly KSv2 labels May 29, 2024
@c3024
Copy link
Contributor

c3024 commented May 30, 2024

@marcaaron Can you please assign me this issue as C+ as I reviewed the PR?

@melvin-bot melvin-bot bot added Weekly KSv2 Awaiting Payment Auto-added when associated PR is deployed to production and removed Weekly KSv2 labels Jun 4, 2024
@melvin-bot melvin-bot bot changed the title Uploading a Group Chat custom avatar that is too large shows badly formatted error message [HOLD for payment 2024-06-11] Uploading a Group Chat custom avatar that is too large shows badly formatted error message Jun 4, 2024
@melvin-bot melvin-bot bot removed the Reviewing Has a PR in review label Jun 4, 2024
Copy link

melvin-bot bot commented Jun 4, 2024

Reviewing label has been removed, please complete the "BugZero Checklist".

Copy link

melvin-bot bot commented Jun 4, 2024

The solution for this issue has been 🚀 deployed to production 🚀 in version 1.4.78-5 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

If no regressions arise, payment will be issued on 2024-06-11. 🎊

For reference, here are some details about the assignees on this issue:

  • @s77rt requires payment (Needs manual offer from BZ)
  • @c3024 requires payment (Needs manual offer from BZ)

Copy link

melvin-bot bot commented Jun 11, 2024

Issue is ready for payment but no BZ is assigned. @laurenreidexpensify you are the lucky winner! Please verify the payment summary looks correct and complete the checklist. Thanks!

@melvin-bot melvin-bot bot removed the Overdue label Jun 11, 2024
Copy link

melvin-bot bot commented Jun 11, 2024

Payment Summary

Upwork Job

  • ROLE: @s77rt paid $(AMOUNT) via Upwork (LINK)
  • ROLE: @c3024 paid $(AMOUNT) via Upwork (LINK)

BugZero Checklist (@laurenreidexpensify)

  • I have verified the correct assignees and roles are listed above and updated the neccesary manual offers
  • I have verified that there are no duplicate or incorrect contracts on Upwork for this job (https://www.upwork.com/ab/applicants//hired)
  • I have paid out the Upwork contracts or cancelled the ones that are incorrect
  • I have verified the payment summary above is correct

@laurenreidexpensify laurenreidexpensify added the External Added to denote the issue can be worked on by a contributor label Jun 12, 2024
@melvin-bot melvin-bot bot changed the title [HOLD for payment 2024-06-11] Uploading a Group Chat custom avatar that is too large shows badly formatted error message [$250] [HOLD for payment 2024-06-11] Uploading a Group Chat custom avatar that is too large shows badly formatted error message Jun 12, 2024
Copy link

melvin-bot bot commented Jun 12, 2024

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

@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Jun 12, 2024
Copy link

melvin-bot bot commented Jun 12, 2024

Current assignees @s77rt and @c3024 are eligible for the External assigner, not assigning anyone new.

@laurenreidexpensify
Copy link
Contributor

@s77rt @c3024 pls apply in upwork thanks - https://www.upwork.com/jobs/~013621cec02e1951b1

@s77rt
Copy link
Contributor

s77rt commented Jun 12, 2024

@laurenreidexpensify Accepted! Thanks!

@c3024
Copy link
Contributor

c3024 commented Jun 13, 2024

@laurenreidexpensify applied, thank you.

@laurenreidexpensify
Copy link
Contributor

Summary:

  • C+ @s77rt paid $250 via Upwork
  • Contributor @c3024 paid $250 via Upwork

@s77rt regression step recommendations for this?

@s77rt
Copy link
Contributor

s77rt commented Jun 13, 2024

@laurenreidexpensify The bug was found during PR review and not entirely a regression. I don't think this is worth adding a test for

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Daily KSv2 External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors
Projects
None yet
Development

No branches or pull requests

6 participants