[$250] Profile - Zoom slider does not change position when moving to zoom an image

[lanitochka17]

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: 9.0.6-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers): gatantm+528@gmail.com
Issue reported by: Applause - Internal Team

Issue found when executing PR #44788

Action Performed:

1. Navigate to account Settings > Profile
2. Click on the avatar
3. Click on upload photo and upload any JPEG image
4. Hover over the zoom slider and move to the right

Expected Result:

The zoom slider is moving to show how an image is zoomed

Actual Result:

The zoom slider stays on the same position when moving it to zoom uploaded image

Workaround:

Unknown

Platforms:

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

• Android: Native
• Android: mWeb Chrome
• iOS: Native
• iOS: mWeb Safari
• MacOS: Chrome / Safari
• MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

Bug6537954_1720620814979.Recording__964.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~0130ac939be52a2a80
• Upwork Job ID: 1811335809081764317
• Last Price Increase: 2024-09-19

[melvin-bot]

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

[github-actions]

👋 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.

[lanitochka17]

@Julesssss FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

[lanitochka17]

We think that this bug might be related to #vip-vsp

[Julesssss]

Cute cat.

This needs to be fixed, but I'm demoting it from being a blocker.

[Julesssss]

My best guess so far is that this PR introduced the bug

[ishpaul777]

I can reproduce the issue even after reverting #44788

[Julesssss]

Thanks for looking

[melvin-bot]

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

[melvin-bot]

Triggered auto assignment to @alexpensify (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]

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

[melvin-bot]

📣 @harshit078! 📣
Hey, it seems we don’t have your contributor details yet! You'll only have to do this once, and this is how we'll hire you on Upwork.
Please follow these steps:

1. Make sure you've read and understood the contributing guidelines.
2. Get the email address used to login to your Expensify account. If you don't already have an Expensify account, create one here. If you have multiple accounts (e.g. one for testing), please use your main account email.
3. Get the link to your Upwork profile. It's necessary because we only pay via Upwork. You can access it by logging in, and then clicking on your name. It'll look like this. If you don't already have an account, sign up for one here.
4. Copy the format below and paste it in a comment on this issue. Replace the placeholder text with your actual details.
   
   Format:

Contributor details
Your Expensify account email: <REPLACE EMAIL HERE>
Upwork Profile Link: <REPLACE LINK HERE>

[QichenZhu]

Proposal

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

The zoom slider doesn't work.

What is the root cause of that problem?

In versions below 2.16.0, react-native-gesture-handler registers pointermove listeners but never unregisters them. This causes issues in development mode since StrictMode renders components twice. This has been fixed by software-mansion/react-native-gesture-handler#2815.

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

1. Wait until [HOLD for payment 2024-07-24] [$250] Profile - App crashes when uploading profile picture #45162 is resolved. Currently I'm using the work-in-progress solution from Make Slider Transformation Function Callback a worklet #45314.
2. Upgrade react-native-gesture-handler to version 2.16.0 or newer.

What alternative solutions did you explore? (Optional)

N/A

[alexpensify]

Weekly Update: #40548

[melvin-bot]

This issue has not been updated in over 15 days. @alexpensify, @Julesssss, @hungvu193 eroding to Monthly issue.

P.S. Is everyone reading this sure this is really a near-term priority? Be brave: if you disagree, go ahead and close it out. If someone disagrees, they'll reopen it, and if they don't: one less thing to do!

[alexpensify]

Weekly update: The other GH we have been tracking states - Closing this in favor of the 0.75 upgrade

I need to figure out what we should do here. @roryabraham any feedback? Thanks!

[roryabraham]

Should HOLD on #37374 instead

[alexpensify]

Weekly Update: Thanks, Rory! Still on hold for #37374

[roryabraham]

We are on RN 0.75 now, so I think this can come off hold. That issue isn't closed yet but is in the cleanup phase.

[alexpensify]

Thank you, @roryabraham, for the update. @hungvu193 please keep us posted if you agree with this plan. Thanks!

[hungvu193]

Agreed!

[hungvu193]

@alexpensify I think this is fixed on latest main, can you re-test this issue?

Screen.Recording.2024-09-05.at.16.02.33.mov

[melvin-bot]

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

[alexpensify]

@lanitochka17 can your team please retest to confirm if there is still an issue? Thanks!

[MelvinBot]

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

[alexpensify]

Weekly update: waiting on a retest

[melvin-bot]

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

[alexpensify]

Weekly Update: I'll need to ask in Slack soon if no movement here.

[melvin-bot]

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

[mvtglobally]

Issue not reproducible during KI retests. (First week)

[alexpensify]

Closing - this is confirmed in the recent test.