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] Mobile browser camera too zoomed in and not focusing. (iPhone Pro Max 16) #51347

Closed
1 of 8 tasks
m-natarajan opened this issue Oct 23, 2024 · 75 comments
Closed
1 of 8 tasks
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. External Added to denote the issue can be worked on by a contributor Reviewing Has a PR in review Weekly KSv2

Comments

@m-natarajan
Copy link

m-natarajan commented Oct 23, 2024

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!


Version Number: 9.0.51-4
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?:
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
Expensify/Expensify Issue URL:
Issue reported by: @ryanschaffer
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1729635637264959

Action Performed:

  1. Open New Expensify in mobile browser (I used incognito tab on chrome on an iphone 16 pro max)
  2. Click green plus
  3. Click Submit Expense
  4. Enable camera permission
  5. Camera is too zoomed in and unable to focus

Expected Result:

The camera is at a normal zoom level and focuses on the text

Actual Result:

The camera is super zoomed in and unable to focus on the text

Workaround:

Unknown

Platforms:

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

  • Android: Standalone
  • Android: HybridApp
  • Android: mWeb Chrome
  • iOS: Standalone
  • iOS: HybridApp
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence
ScreenRecording_10-23-2024.19-10-39_1.MP4

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~021852190537586837285
  • Upwork Job ID: 1852190537586837285
  • Last Price Increase: 2024-12-13
Issue OwnerCurrent Issue Owner: @alitoshmatov
@m-natarajan m-natarajan added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Oct 23, 2024
Copy link

melvin-bot bot commented Oct 23, 2024

Triggered auto assignment to @RachCHopkins (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.

@RachCHopkins
Copy link
Contributor

Opened an incognito window on iOS chrome and totally unable to tap into the email box. Tapped once and it says "please enter an email or phone number" in red. I am unable to tap to get the keyboard. Same non-incognito.

Got in on Safari.

Zoom is totally normal on iPhone 14 on iOS 18.0.1

@RachCHopkins
Copy link
Contributor

@m-natarajan is your team able to reproduce this?

@RachCHopkins RachCHopkins added the Needs Reproduction Reproducible steps needed label Oct 24, 2024
@MelvinBot
Copy link

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

@melvin-bot melvin-bot bot added the Overdue label Oct 28, 2024
@RachCHopkins
Copy link
Contributor

Waiting on repro.

@melvin-bot melvin-bot bot removed the Overdue label Oct 29, 2024
@RachCHopkins
Copy link
Contributor

Requested help here

@RachCHopkins
Copy link
Contributor

Applause has replicated this and said it's specific to mobile Chrome and this iPhone version.

@RachCHopkins RachCHopkins added Help Wanted Apply this label when an issue is open to proposals by contributors External Added to denote the issue can be worked on by a contributor and removed Needs Reproduction Reproducible steps needed Help Wanted Apply this label when an issue is open to proposals by contributors labels Nov 1, 2024
Copy link

melvin-bot bot commented Nov 1, 2024

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

@melvin-bot melvin-bot bot changed the title Mobile browser camera too zoomed in and not focusing. (iPhone Pro Max 16) [$250] Mobile browser camera too zoomed in and not focusing. (iPhone Pro Max 16) Nov 1, 2024
@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Nov 1, 2024
Copy link

melvin-bot bot commented Nov 1, 2024

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

Copy link

melvin-bot bot commented Nov 4, 2024

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

@melvin-bot melvin-bot bot added the Overdue label Nov 4, 2024
@RachCHopkins
Copy link
Contributor

No proposals yet!

@melvin-bot melvin-bot bot removed the Overdue label Nov 5, 2024
Copy link

melvin-bot bot commented Nov 8, 2024

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

@melvin-bot melvin-bot bot added the Overdue label Nov 8, 2024
@alitoshmatov
Copy link
Contributor

No proposals, also can't reproduce as it requires physical iphone 16

@melvin-bot melvin-bot bot removed the Overdue label Nov 10, 2024
Copy link

melvin-bot bot commented Dec 13, 2024

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

@fabioh8010
Copy link
Contributor

Working on this right now Melvin

@fabioh8010
Copy link
Contributor

Having problems with building for iOS device, will continue tomorrow

@mvtglobally
Copy link

Issue not reproducible during KI retests. (First week)

@fabioh8010
Copy link
Contributor

Tests done, asked @pac-guerreiro to edit the PR content and open it to review since he's the author.

@fabioh8010
Copy link
Contributor

FYI I will be heading to vacations and will return on January 7th, so we will find someone (preferably with a 3-camera device) in our team to take over this issue from me. @pac-guerreiro will also help with updating the PR description if necessary.

Copy link

melvin-bot bot commented Dec 16, 2024

@RachCHopkins, @alitoshmatov Now this issue is 8 days overdue. Are you sure this should be a Daily? Feel free to change it!

@melvin-bot melvin-bot bot added Reviewing Has a PR in review Weekly KSv2 and removed Daily KSv2 Overdue labels Dec 16, 2024
@alitoshmatov
Copy link
Contributor

I will confirm this!

Oh, I assume you meant that you do confirm.

We will have to get approval of internal engineer before proceeding to PR

@alitoshmatov
Copy link
Contributor

We decided to implement this proposal from @fabioh8010 but it looks like someone else will be taking over, so not sure whom to assign

C+ reviewed 🎀 👀 🎀

Copy link

melvin-bot bot commented Dec 16, 2024

Triggered auto assignment to @deetergp, see https://stackoverflow.com/c/expensify/questions/7972 for more details.

@pac-guerreiro
Copy link
Contributor

@RachCHopkins I applied @fabioh8010 updates to the PR description, removed the [WIP] label and opened it to review 😄

@melvin-bot melvin-bot bot removed the Help Wanted Apply this label when an issue is open to proposals by contributors label Dec 17, 2024
@mvtglobally
Copy link

Issue not reproducible during KI retests. (Second week)

@deetergp
Copy link
Contributor

If the issue has not been reproducible for the last two weeks, is this PR still necessary?

@pac-guerreiro
Copy link
Contributor

@deetergp I'll need to wait for @fabioh8010 to come back next week and confirm it

@fabioh8010
Copy link
Contributor

I'm going to test again today!

@fabioh8010
Copy link
Contributor

It seems that the issue is not reproducible anymore (tested on Production). cc @deetergp @pac-guerreiro @alitoshmatov

ScreenRecording_01-07-2025.16-26-45_1-compressed.MP4
ScreenRecording_01-07-2025.16-27-39_1-compressed.MP4

@alitoshmatov
Copy link
Contributor

Weird, seems like nothing has changed in https://github.com/Expensify/App/blob/main/src/pages/iou/request/step/IOURequestStepScan/index.tsx

I think we can hold the PR and wait couple of days to retest this

@deetergp
Copy link
Contributor

deetergp commented Jan 8, 2025

Alright. Let's retest it on Friday, and if it still isn't reproducible, let's close it out.

@deetergp
Copy link
Contributor

It's Friday! 🎉 Time to give this one another test @alitoshmatov?

@fabioh8010
Copy link
Contributor

Just tested it now and still not reproducible anymore (tested on Production). cc @deetergp @pac-guerreiro @alitoshmatov

ScreenRecording_01-13-2025.09-25-35_1-compressed.MP4
ScreenRecording_01-13-2025.09-26-41_1-compressed.MP4

@deetergp
Copy link
Contributor

Since this already seems to have been resolved in prod, we are going to close this one.

@github-project-automation github-project-automation bot moved this from Bugs and Follow Up Issues to Done in [#whatsnext] #expense Jan 16, 2025
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. External Added to denote the issue can be worked on by a contributor Reviewing Has a PR in review Weekly KSv2
Projects
Status: Done
8 participants