Skip to content

[$250] iOS Hybrid - Lack of debug mode in the settings - Troubleshoot for IOS Hybrid app #50843

Closed

Description

@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.49-0
Reproducible in staging?: Y
Reproducible in production?: Y
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): applausetester+en@applause.expensifail.com
Issue reported by: Applause - Internal Team

Issue found when executing PR #50519

Action Performed:

  1. Open the Expenisfy Hybrid app
  2. Sign into a valid account and access the ND features within the app
  3. Go to Account section > Troubleshoot > Scroll down

Expected Result:

User expects to see the Debug option

Actual Result:

No Debug option is shown

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
Bug6635787_1729028252122.No_Debug_option_in_Troubleshoot_menu_.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~021846335027010160243
  • Upwork Job ID: 1846335027010160243
  • Last Price Increase: 2024-10-29
Issue OwnerCurrent Issue Owner: @situchan

Activity

  1. added
    BugSomething is broken. Auto assigns a BugZero manager.
    on Oct 15, 2024
  2. melvin-bot commented on Oct 15, 2024

    @melvin-bot

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

  3. lanitochka17 commented on Oct 15, 2024

    @lanitochka17
    Author

    @greg-schroeder 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

  4. added
    ExternalAdded to denote the issue can be worked on by a contributor
    on Oct 15, 2024
  5. melvin-bot commented on Oct 15, 2024

    @melvin-bot
  6. changed the title [-]iOS Hybrid - Lack of debug mode in the settings - Troubleshoot for IOS Hybrid app[/-] [+][$250] iOS Hybrid - Lack of debug mode in the settings - Troubleshoot for IOS Hybrid app[/+] on Oct 15, 2024
  7. added
    Help WantedApply this label when an issue is open to proposals by contributors
    on Oct 15, 2024
  8. melvin-bot commented on Oct 15, 2024

    @melvin-bot

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

  9. layacat commented on Oct 16, 2024

    @layacat
    Contributor

    Proposal

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

    Lack of debug mode in the settings - Troubleshoot for IOS Hybrid app

    What is the root cause of that problem?

    We always return Production for HybridAppModule here:

    if (NativeModules.HybridAppModule) {
    environment = CONST.ENVIRONMENT.PRODUCTION;
    return;
    }

    That's the RCA.

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

    We should remove this block of code:

     if (NativeModules.HybridAppModule) { 
         environment = CONST.ENVIRONMENT.PRODUCTION; 
         return; 
     } 
    

    What alternative solutions did you explore? (Optional)

    N/A
    Reminder: Please use plain English, be brief and avoid jargon. Feel free to use images, charts or pseudo-code if necessary. Do not post large multi-line diffs or write walls of text. Do not create PRs unless you have been hired for this job.

  10. situchan commented on Oct 16, 2024

    @situchan
    Contributor

    can we allow debug mode in hybrid app?
    tagging hybrid app experts: @AndrewGable @staszekscp @mateuuszzzzz

  11. AndrewGable commented on Oct 16, 2024

    @AndrewGable
    Contributor

    I believe we can, but let's confirm with @staszekscp and @mateuuszzzzz

  12. 21 remaining items

  13. Julesssss commented on Oct 24, 2024

    @Julesssss
    Contributor

    Hey @blazejkustra could you comment here and I'll assign you, thanks!

  14. self-assigned this
    on Oct 24, 2024
  15. melvin-bot commented on Oct 29, 2024

    @melvin-bot

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

  16. greg-schroeder commented on Oct 29, 2024

    @greg-schroeder
    Contributor

    Okay I'm a bit confused on this one - who is going to take this one on? Or are we still waiting for a valid proposal?

  17. Julesssss commented on Oct 29, 2024

    @Julesssss
    Contributor

    I think we can close this once this issue is completed (its awaiting testing on prod)

  18. greg-schroeder commented on Nov 1, 2024

    @greg-schroeder
    Contributor

    Held til 2024-11-07

  19. melvin-bot commented on Nov 5, 2024

    @melvin-bot

    @Julesssss, @greg-schroeder, @situchan Whoops! This issue is 2 days overdue. Let's get this updated quick!

  20. Julesssss commented on Nov 6, 2024

    @Julesssss
    Contributor

    Still held

  21. Julesssss commented on Nov 8, 2024

    @Julesssss
    Contributor

    The linked issue is closed. We didn't end up needing C+ review here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

BugSomething is broken. Auto assigns a BugZero manager.DailyKSv2ExternalAdded to denote the issue can be worked on by a contributorHelp WantedApply this label when an issue is open to proposals by contributorsReviewingHas a PR in review

Type

No type

Projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions