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 for payment 2022-05-02] [$1000] When you tap on any image/attachment and scroll, it fades - Reported by @mdneyazahmad #7596

Closed
mvtglobally opened this issue Feb 7, 2022 · 51 comments
Assignees
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Daily KSv2 Engineering External Added to denote the issue can be worked on by a contributor Reviewing Has a PR in review

Comments

@mvtglobally
Copy link

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 app
  2. Navigate to any chat with attachments
  3. Click on attachment and scroll

Expected Result:

Image should be visible

Actual Result:

When you tap on image and scroll, It fades

Workaround:

unknown

Platform:

Where is this issue occurring?

  • Web
  • iOS
  • Android
  • Desktop App
  • Mobile Web

Version Number: 1.1.35-0
Reproducible in staging?: Y
Reproducible in production?: Y
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

Record_2022-01-29-19-32-12.mp4

Expensify/Expensify Issue URL:
Issue reported by: @mdneyazahmad
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1643465045973799

View all open jobs on GitHub

@mvtglobally mvtglobally added AutoAssignerTriage Auto assign issues for triage to an available triage team member Daily KSv2 labels Feb 7, 2022
@MelvinBot
Copy link

Triggered auto assignment to @flaviadefaria (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

@MelvinBot MelvinBot added Overdue and removed AutoAssignerTriage Auto assign issues for triage to an available triage team member labels Feb 7, 2022
@flaviadefaria
Copy link
Contributor

Doesn't seem like a big issue or a very high priority, but the bug does exist so adding the engineering label.

@MelvinBot
Copy link

Triggered auto assignment to @marcochavezf (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

@flaviadefaria flaviadefaria removed their assignment Feb 11, 2022
@MelvinBot
Copy link

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

2 similar comments
@MelvinBot
Copy link

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

@MelvinBot
Copy link

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

@MelvinBot
Copy link

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

2 similar comments
@MelvinBot
Copy link

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

@MelvinBot
Copy link

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

@MelvinBot
Copy link

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

@MelvinBot
Copy link

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

@MelvinBot
Copy link

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

1 similar comment
@MelvinBot
Copy link

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

@marcochavezf
Copy link
Contributor

Oops MelvinBot is a little anxious about this one 😄
I was able to reproduce it on web too. Applying the External label and Weekly because seems to be an edge-case

@MelvinBot MelvinBot removed the Overdue label Feb 14, 2022
@marcochavezf marcochavezf removed their assignment Feb 14, 2022
@marcochavezf marcochavezf added Weekly KSv2 External Added to denote the issue can be worked on by a contributor and removed Daily KSv2 labels Feb 14, 2022
@mdneyazahmad
Copy link
Contributor

@jliexpensify https://www.upwork.com/freelancers/~011922ae9ff280b160 Thank you!

@kadiealexander kadiealexander removed their assignment Apr 1, 2022
@melvin-bot melvin-bot bot added the Overdue label Apr 8, 2022
@jliexpensify
Copy link
Contributor

Not overdue. @mdneyazahmad how are you going here? :)

@melvin-bot melvin-bot bot removed the Overdue label Apr 11, 2022
@mdneyazahmad
Copy link
Contributor

@jliexpensify Pr is about to be merged.

@chiragsalian chiragsalian added the Reviewing Has a PR in review label Apr 14, 2022
@jliexpensify
Copy link
Contributor

Reassigning as I'm OOO for 2 weeks: it looks like payment will need to be made within that time frame.

Internal - https://www.upwork.com/ab/applicants/1493422908376997888/job-details
External - https://www.upwork.com/jobs/~01ed5925c98421d1c6

@jliexpensify jliexpensify removed their assignment Apr 14, 2022
@jliexpensify jliexpensify added External Added to denote the issue can be worked on by a contributor and removed External Added to denote the issue can be worked on by a contributor labels Apr 14, 2022
@melvin-bot
Copy link

melvin-bot bot commented Apr 14, 2022

Triggered auto assignment to @stephanieelliott (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

@melvin-bot melvin-bot bot added Daily KSv2 and removed Weekly KSv2 labels Apr 14, 2022
@stephanieelliott
Copy link
Contributor

PR is undergoing QA, not merged to staging yet.

@stephanieelliott
Copy link
Contributor

PR on staging

@melvin-bot melvin-bot bot added Weekly KSv2 Awaiting Payment Auto-added when associated PR is deployed to production and removed Daily KSv2 labels Apr 25, 2022
@melvin-bot melvin-bot bot changed the title [$1000] When you tap on any image/attachment and scroll, it fades - Reported by @mdneyazahmad [HOLD for payment 2022-05-02] [$1000] When you tap on any image/attachment and scroll, it fades - Reported by @mdneyazahmad Apr 25, 2022
@melvin-bot
Copy link

melvin-bot bot commented Apr 25, 2022

The solution for this issue has been 🚀 deployed to production 🚀 in version 1.1.56-0 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 2022-05-02. 🎊

@melvin-bot melvin-bot bot added Daily KSv2 and removed Weekly KSv2 Daily KSv2 labels May 2, 2022
@stephanieelliott
Copy link
Contributor

All paid!

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 Engineering External Added to denote the issue can be worked on by a contributor Reviewing Has a PR in review
Projects
None yet
Development

No branches or pull requests