Skip to content

[$500] IOS - Request - Camera blinks when switching between tabs  #30280

Closed
Closed
[$500] IOS - Request - Camera blinks when switching between tabs #30280
Assignees
dangroussonialiapshubham1206agra
Labels
BugSomething is broken. Auto assigns a BugZero manager.DailyKSv2EngineeringExternalAdded to denote the issue can be worked on by a contributorHelp WantedApply this label when an issue is open to proposals by contributors

Description

@lanitochka17
lanitochka17
opened on Oct 24, 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!

Version Number: 1.3.90-1

Reproducible in staging?: Yes

Reproducible in production?: No

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: Applause - Internal Team

Slack conversation:

Issue found when executing PR #28792

Action Performed:

  1. Go to + > Request money
  2. Go to Scan and enable camera access if not allowed yet
  3. Go to Distance, then Manual a few times
  4. Go back to Scan

Expected Result:

The camera should not blink when switching between tabs

Actual Result:

Camera blinks when switching between tabs

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

Android: Native
Android: mWeb Chrome
iOS: Native

Staging:

Bug6249100_1698167102988.0-02-01-c1ba15d79bd9b53cf3056528c8a8e5eb86b1e7520a79a08e5f979466bc08cde3_4846e703b4ace5fc__1_.mp4

Production:

0-02-01-77ed3d9a6859e1c1b298814e92bf3c0ec9cc073350773a18f2731b529f70c7b5_e98c343e15be4769.1.mp4
iOS: mWeb Safari
MacOS: Chrome / Safari
MacOS: Desktop

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~019e1abb1d81693a85
  • Upwork Job ID: 1717191476381212672
  • Last Price Increase: 2023-11-01

Activity

  1. lanitochka17
    added
    DeployBlockerCashThis issue or pull request should block deployment
    DailyKSv2
    BugSomething is broken. Auto assigns a BugZero manager.
    on Oct 24, 2023
  2. melvin-bot
    assigned
    sonialiap
    on Oct 24, 2023

  3. melvin-bot commented on Oct 24, 2023

    @melvin-bot
    melvin-botBot
    on Oct 24, 2023

    Triggered auto assignment to @sonialiap (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

  4. melvin-bot commented on Oct 24, 2023

    @melvin-bot
    melvin-botBot
    on Oct 24, 2023

    Bug0 Triage Checklist (Main S/O)

    • This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
    • This bug is not a duplicate report (check E/App issues and #expensify-bugs)
      • If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead
    • This bug is reproducible using the reproduction steps in the OP. S/O
      • If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
      • If the reproduction steps aren't clear and you determine the correct steps, please update the OP.
    • This issue is filled out as thoroughly and clearly as possible
      • Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.
    • I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync
  5. github-actions
    added
    Engineering
    HourlyKSv2
    and removed
    DailyKSv2
    on Oct 24, 2023

  6. OSBotify commented on Oct 24, 2023

    @OSBotify
    OSBotify
    on Oct 24, 2023
    Contributor

    👋 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.
  7. melvin-bot
    assigned
    dangrous
    on Oct 24, 2023

  8. melvin-bot commented on Oct 24, 2023

    @melvin-bot
    melvin-botBot
    on Oct 24, 2023

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

  9. ishpaul777 commented on Oct 24, 2023

    @ishpaul777
    ishpaul777
    on Oct 24, 2023
    Contributor

    maybe a regression from #28792

  10. lanitochka17 commented on Oct 24, 2023

    @lanitochka17
    lanitochka17
    on Oct 24, 2023
    Author

    @ishpaul777 You are right. Issue was found when executing PR #28792

  11. ishpaul777 commented on Oct 24, 2023

    @ishpaul777
    ishpaul777
    on Oct 24, 2023
    Contributor

    I can further investigate the root cause, but that would require app to builld on physical device (simulator dont have camera) Is there a resource I can follow to build app on Ios device I tried and failed to build in past, a few times, cc @dangrous

  12. 9 remaining items

  13. melvin-bot commented on Oct 25, 2023

    @melvin-bot
    melvin-botBot
    on Oct 25, 2023

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

  14. melvin-bot
    added
    Help WantedApply this label when an issue is open to proposals by contributors
    on Oct 25, 2023
  15. melvin-bot
    assigned
    shubham1206agra
    on Oct 25, 2023

  16. melvin-bot commented on Oct 25, 2023

    @melvin-bot
    melvin-botBot
    on Oct 25, 2023

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

  17. mountiny commented on Oct 25, 2023

    @mountiny
    mountiny
    on Oct 25, 2023
    Contributor

    I agree this is not a blocker, I have seen the blink but I have seen that in previous versions too @dukenv0307 I think this should be handled as a regression from your PR though as it apprently introduced it cc @sonialiap

  18. melvin-bot
    added
    Overdue
    on Oct 30, 2023

  19. sonialiap commented on Oct 30, 2023

    @sonialiap
    sonialiap
    on Oct 30, 2023
    Contributor

    @mountiny we'll want to fix this in #27879 and then pay the reporter, is that correct?

  20. melvin-bot
    removed
    Overdue
    on Oct 30, 2023

  21. dangrous commented on Oct 31, 2023

    @dangrous
    dangrous
    on Oct 31, 2023
    Contributor

    Yeah I think ideally @dukenv0307 and @mananjadhav would take care of this as a regression (or at least an improvement) from #27879 - let me know if you think differently @mountiny

  22. mananjadhav commented on Oct 31, 2023

    @mananjadhav
    mananjadhav
    on Oct 31, 2023
    Collaborator

    I agree we can take this as an improvement from the previous PR. But we would still need help to test it on IOS.

    @dukenv0307 can we please work on the follow up PR?

  23. situchan commented on Oct 31, 2023

    @situchan
    situchan
    on Oct 31, 2023
    Contributor

    I can help testing. I can install app on iOS physical device

  24. mountiny commented on Nov 1, 2023

    @mountiny
    mountiny
    on Nov 1, 2023
    Contributor

    I agree this should be treated as a regression and so we could also close this issue, pay the reporter first but handle this in the original issue

  25. melvin-bot commented on Nov 1, 2023

    @melvin-bot
    melvin-botBot
    on Nov 1, 2023 – with Melvin Bot

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

  26. dangrous commented on Nov 1, 2023

    @dangrous
    dangrous
    on Nov 1, 2023
    Contributor

    Okay cool, that works for me. Closing! I just commented on the main issue with these details.

  27. dangrous
    closed this as completedon Nov 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

BugSomething is broken. Auto assigns a BugZero manager.DailyKSv2EngineeringExternalAdded to denote the issue can be worked on by a contributorHelp WantedApply this label when an issue is open to proposals by contributors

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions