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

[HOLD App issue #13555] Avatar preview is blurry #14349

Closed
6 tasks
kavimuru opened this issue Jan 17, 2023 · 15 comments
Closed
6 tasks

[HOLD App issue #13555] Avatar preview is blurry #14349

kavimuru opened this issue Jan 17, 2023 · 15 comments
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Engineering Weekly KSv2

Comments

@kavimuru
Copy link

kavimuru commented Jan 17, 2023

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


Action Performed:

  1. Open the app
  2. Go to Settings => Profile => Change the Avatar with any images that doesn’t look blurry.
  3. Kill the app and wait for a while, open the app again and try to preview your avatar inside any report.
  4. Notice that your avatar preview looks so blurry.

Expected Result:

Avatar preview shouldn’t look blurry

Actual Result:

Avatar preview looks blurry.

Workaround:

unknown

Platforms:

Which of our officially supported platforms is this issue occurring on?

  • Android / native
  • Android / Chrome
  • iOS / native
  • iOS / Safari
  • MacOS / Chrome / Safari
  • MacOS / Desktop

Version Number: 1.2.54-2
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:

repro.1601.mWeb.mp4
RPReplay_Final1673615422.1.MP4
repro.1601.android.mp4

Expensify/Expensify Issue URL:
Issue reported by: @hungvu193
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1673615491816959

View all open jobs on GitHub

@kavimuru kavimuru added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Jan 17, 2023
@melvin-bot melvin-bot bot locked and limited conversation to collaborators Jan 17, 2023
@lschurr
Copy link
Contributor

lschurr commented Jan 17, 2023

I don't think the second video in the OP is relevant right @kavimuru?

@lschurr
Copy link
Contributor

lschurr commented Jan 17, 2023

Hm, I wasn't able to reproduce this. I changed my avatar image, killed the app, went back later and it still looks clear. Adding Eng for more eyes.

@kavimuru
Copy link
Author

@lschurr Updated video. Sorry about that.

@melvin-bot melvin-bot bot added the Overdue label Jan 19, 2023
@cristipaval
Copy link
Contributor

@Beamanator I think this should be fixed by the fix for your issue. Do you agree?

@melvin-bot melvin-bot bot removed the Overdue label Jan 19, 2023
@Beamanator
Copy link
Contributor

Aah yes most likely, because we're full-screening a thumbnail picture :D

@cristipaval
Copy link
Contributor

cristipaval commented Jan 20, 2023

Thanks for the confirmation. I'll set this on hold for now and test it again after your fix.

@cristipaval cristipaval changed the title Avatar preview is blurry. [HOLD App issue #13555] Avatar preview is blurry Jan 20, 2023
@melvin-bot melvin-bot bot added the Overdue label Jan 23, 2023
@cristipaval
Copy link
Contributor

Weekly - same as the issue that is held on.

@melvin-bot melvin-bot bot removed the Overdue label Jan 23, 2023
@cristipaval cristipaval added Weekly KSv2 and removed Daily KSv2 labels Jan 23, 2023
@melvin-bot melvin-bot bot added the Overdue label Feb 1, 2023
@lschurr
Copy link
Contributor

lschurr commented Feb 1, 2023

Still on hold.

@melvin-bot melvin-bot bot removed the Overdue label Feb 1, 2023
@melvin-bot melvin-bot bot added the Overdue label Feb 9, 2023
@lschurr
Copy link
Contributor

lschurr commented Feb 9, 2023

Still on hold.

@melvin-bot melvin-bot bot removed the Overdue label Feb 9, 2023
@melvin-bot melvin-bot bot added the Overdue label Feb 20, 2023
@lschurr
Copy link
Contributor

lschurr commented Feb 20, 2023

Believe this is still on hold.. is that right @cristipaval?

@melvin-bot melvin-bot bot removed the Overdue label Feb 20, 2023
@cristipaval
Copy link
Contributor

Hmm.. it looks like the fix for the other one was merged. I'll test this one today and keep you posted here.

@cristipaval
Copy link
Contributor

yeah, it looks like this one is fixed now by this PR

@lschurr
Copy link
Contributor

lschurr commented Feb 21, 2023

Reopening so that we can issue a reporting bonus - https://expensify.slack.com/archives/C01GTK53T8Q/p1676940741190779

@lschurr lschurr reopened this Feb 21, 2023
@lschurr
Copy link
Contributor

lschurr commented Feb 21, 2023

@hungvu193 - Could you apply for the job in Upwork? https://www.upwork.com/jobs/~01903124f5eb7845ca

@lschurr
Copy link
Contributor

lschurr commented Feb 21, 2023

All set - payment has been issued.

@lschurr lschurr closed this as completed Feb 21, 2023
@Expensify Expensify unlocked this conversation Mar 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. Engineering Weekly KSv2
Projects
None yet
Development

No branches or pull requests

4 participants