[HOLD for payment 2022-11-09] [$250] Web - Clicking on zoom photo line should zoom the photo and the large green button should be place at clicked point reported by @aneequeahmad

[kavimuru]

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. Click on use icon to open setting page.
2. Click on any workspace > general settings > edit photo > upload photo.
3. Click on the the zoom line

Expected Result:

Should zoom image and drag the green button to clicked point

Actual Result:

Not zooming image and dragging the green button.

Workaround:

Unknown

Platform:

Where is this issue occurring?

• Web

Version Number: v1.1.99-4
Reproducible in staging?: Y
Reproducible in production?: Y
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
https://user-images.githubusercontent.com/43996225/189779379-a51506f4-1646-4f9f-a79b-9b946af9e27e.mp4
Expensify/Expensify Issue URL:
Issue reported by: @aneequeahmad
Slack conversation:

View all open jobs on GitHub

[melvin-bot]

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

[melvin-bot]

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

[varshamb]

Proposal

We should use Component Slider from '@react-native-community/slider'
I will give detailed proposal if it is okay to use this.

[tgolen]

@varshamb We typically don't use external components since most things can be built pretty easily and specifically just for our use case. Could you make a proposal that is an improvement to our current <Slider> instead?

[laurenreidexpensify]

Can confirm that this is reproducible.

[melvin-bot]

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

[laurenreidexpensify]

@tgolen is this good to get the exported label?

[melvin-bot]

Current assignee @laurenreidexpensify is eligible for the External assigner, not assigning anyone new.

[laurenreidexpensify]

https://www.upwork.com/jobs/~0117a7dd51bf4a5121

[laurenreidexpensify]

heh why isn't this adding a C+

[thesahindia]

Maybe because we don't have the help wanted label here

[mallenexpensify]

@laurenreidexpensify co-assigning cuz I'm part of
[Tracking] [Polish] Image-related feature request and improvements #10894

[mananjadhav]

@Abdallah-Ahmed-moh quick bump on this PR comment

@Abdallah-Ahmed-moh Got a chance to look at this comment?

[mallenexpensify]

@Beamanator This job is in progress so I'm not putting on hold pending #10894.
Comment if you disagree

[Beamanator]

Agreed @mallenexpensify 👍

[melvin-bot]

BugZero Checklist: The PR fixing this issue has been merged! The following checklist will need to be completed before the issue can be closed:

• A regression test has been added or updated so that the same bug will not reach production again. Link to the updated test here:
• The PR that introduced the bug has been identified. Link to the PR:
• The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:
• A discussion in #contributor-plus has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:
• Payment has been made to the issue reporter (if applicable)
• Payment has been made to the contributor that fixed the issue (if applicable)
• Payment has been made to the contributor+ that helped on the issue (if applicable)

[laurenreidexpensify]

@Abdallah-Ahmed-moh can you confirm your name in Upwork for me to send you the hiring contract? Thanks

[melvin-bot]

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

• Repair clicking on zoom photo line should zoom the photo #11717

If no regressions arise, payment will be issued on 2022-11-09. 🎊

[laurenreidexpensify]

@mananjadhav @Abdallah-Ahmed-moh @aneequeahmad apologies, there was an issue with the Upwork job posting.

The new post is here https://www.upwork.com/jobs/~0135a64425b4d685b1

Please apply and I will issue payment as soon as you have signed the contract

[aneequeahmad]

@laurenreidexpensify I have applied, thanks for mentioning

[mananjadhav]

Applied @laurenreidexpensify

[laurenreidexpensify]

Still waiting on @Abdallah-Ahmed-moh to apply in Upwork, everyone else has been paid

[Abdallah-Ahmed-moh]

Ok i have apply on upwork on the new post..

[mananjadhav]

@laurenreidexpensify Quick bump on the above comment, so that we can get this one closed out.

[laurenreidexpensify]

@Abdallah-Ahmed-moh can you please send me a link to your Upwork profile so I can confirm I am paying the correct person? Thanks

[Abdallah-Ahmed-moh]

@laurenreidexpensify the upwork is working for my Identity verification , i will tell you while finish

[Abdallah-Ahmed-moh]

@Abdallah-Ahmed-moh can you please send me a link to your Upwork profile so I can confirm I am paying the correct person? Thanks

https://www.upwork.com/freelancers/~01d7c9c17e7faeaa3e

this is my profile, sorry for being late.

[mananjadhav]

@laurenreidexpensify Can you help with the above here and closure of the issue?

[laurenreidexpensify]

@Abdallah-Ahmed-moh contract sent in Upwork, please accept so I can proceed with payment! Thanks

[Abdallah-Ahmed-moh]

@laurenreidexpensify i accepted your offer.

[laurenreidexpensify]

Everyone has been paid 🎉 Closing