Skip to content

[HOLD for payment 2023-10-09] [$500] Dev: Request Money - The app crashes when clicking on the currency option #28317

Closed

Description

@kbecciv

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. Open a chat
  2. Go to the "Request money"
  3. Click on the currency option

Expected Result:

The app should not crash

Actual Result:

The app crashes when clicking on the currency option.

Workaround:

Unknown

Platforms:

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

  • Android / native -requested
  • Android / Chrome -requested
  • iOS / native
  • iOS / Safari
  • MacOS / Chrome / Safari
  • MacOS / Desktop

Version Number: Dev
Reproducible in staging?:
Reproducible in production?:
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
Notes/Photos/Videos: Any additional supporting documentation

Screen.Recording.2023-09-27.at.12.19.03.PM.mov
Screen.Recording.2023-09-27.at.12.15.09.PM.mov
Screen.Recording.2023-09-27.at.12.13.02.PM.mov
screen-recording-2023-09-27-at-113442-am_X1UAbmMm.mp4

Expensify/Expensify Issue URL:
Issue reported by: ayazhussain79
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1695796696023189

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~011cb9c04584439771
  • Upwork Job ID: 1707041619621773312
  • Last Price Increase: 2023-09-27
  • Automatic offers:
    • situchan | Contributor | 27157044

Activity

  1. added
    ExternalAdded to denote the issue can be worked on by a contributor
    on Sep 27, 2023
  2. changed the title [-]Dev: Request Money - The app crashes when clicking on the currency option[/-] [+][$500] Dev: Request Money - The app crashes when clicking on the currency option[/+] on Sep 27, 2023
  3. melvin-bot commented on Sep 27, 2023

    @melvin-bot
  4. added
    Help WantedApply this label when an issue is open to proposals by contributors
    on Sep 27, 2023
  5. melvin-bot commented on Sep 27, 2023

    @melvin-bot

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

  6. hungvu193 commented on Sep 27, 2023

    @hungvu193
    Contributor

    It seems regression from this:
    #27850
    Screen Shot 2023-09-27 at 21 45 23

    This line introduced the bugs, the currency option don't have the icons, we should have safety check there
    cc @bernhardoj
    Screen Shot 2023-09-27 at 21 46 37

  7. bernhardoj commented on Sep 27, 2023

    @bernhardoj
    Contributor

    @hungvu193 yes, you're right. Thanks for the help. I have created a PR to fix it.

  8. mananjadhav commented on Sep 27, 2023

    @mananjadhav
    Collaborator

    I don't see any BZ member added. @bernhardoj Can you assign the original C+ here for the review? or another C+? Because I am afk right now.

  9. bernhardoj commented on Sep 27, 2023

    @bernhardoj
    Contributor

    I can't assign people, so tagging @narefyev91 for visibility. I have tagged him on the PR too.

  10. hungvu193 commented on Sep 27, 2023

    @hungvu193
    Contributor

    That's pretty weird that Melvin added the weekly label, I think that's why there's no BZ assignment 😂

  11. 30 remaining items

  12. situchan commented on Nov 9, 2023

    @situchan
    Contributor

    @sonialiap I was not involved in original issue/PR which caused this regression, so I am eligible.

    @sonialiap bump ^

  13. sonialiap commented on Nov 13, 2023

    @sonialiap
    Contributor

    Offer sent withdrawn in favor of Offer link. Payment made for $500

  14. situchan commented on Nov 14, 2023

    @situchan
    Contributor

    I already got offer here

  15. sonialiap commented on Nov 14, 2023

    @sonialiap
    Contributor

    Payment sent ✔️

  16. ayazhussain79 commented on Nov 14, 2023

    @ayazhussain79
    Contributor

    @sonialiap could you please check am i eligible for reporting bonus?

  17. sonialiap commented on Nov 17, 2023

    @sonialiap
    Contributor

    Checking!

  18. sonialiap commented on Nov 17, 2023

    @sonialiap
    Contributor

    @ayazhussain79 Ah yes of course! Thank you for bringing it up 👍 Offer sent

    Reporting $50 - paid ✔️

  19. ayazhussain79 commented on Nov 17, 2023

    @ayazhussain79
    Contributor

    @sonialiap offer accepted, Thank you

  20. sonialiap commented on Nov 20, 2023

    @sonialiap
    Contributor

    Paid ✔️

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

Metadata

Metadata

Labels

Awaiting PaymentAuto-added when associated PR is deployed to productionBugSomething is broken. Auto assigns a BugZero manager.DailyKSv2ExternalAdded to denote the issue can be worked on by a contributor

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions