Skip to content

[HOLD for payment 2023-12-07] [HOLD for payment 2023-12-06] IOU - Clicking on card and amount does not cause any action #32077

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: 1.4.4-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:
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:

Action Performed:

  1. Go to https://staging.new.expensify.com/
  2. Tap on 1:1 report
  3. Tap plus icon near compose
  4. Tap request money
  5. Enter an amount
  6. Tap next
  7. Tap show more and note "card" option is not displayed
  8. Tap request
  9. Tap on manual request created to open iou detail page
  10. Tap card option

Expected Result:

Tapping on card must trigger an action or card option should not be displayed

Actual Result:

In manual request IOU detail page, tapping on "card" does not trigger any action

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

Bug6292395_1701121963215.oip.mp4

View all open jobs on GitHub

Issue OwnerCurrent Issue Owner: @anmurali

Activity

  1. github-actions commented on Nov 27, 2023

    @github-actions
    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.
  2. melvin-bot commented on Nov 27, 2023

    @melvin-bot

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

  3. marcaaron commented on Nov 28, 2023

    @marcaaron
    Contributor

    Tap show more and note "card" option is not displayed

    Was this a new feature? What is the "card" option?

  4. marcaaron commented on Nov 28, 2023

    @marcaaron
    Contributor

    Tapping on card must trigger an action or card option should not be displayed

    also, what is "action"?

  5. marcaaron commented on Nov 28, 2023

    @marcaaron
    Contributor

    Ohhh you are saying that this should not be here? Got it.

    2023-11-27_14-08-55

    I'd probably phrase this like:

    "Card" option row appears in request detail for regular non-Card expense

  6. marcaaron commented on Nov 28, 2023

    @marcaaron
    Contributor

    Ah we are returning the cash card for the cardID. We will want to make sure that gets omitted and never returned with the report's transactions maybe?

  7. marcaaron commented on Nov 28, 2023

    @marcaaron
    Contributor

    cc @luacmartins I think probably we want to check if the card is "cash" in Auth at some point

    We can revert @grgia's PR here for now to fix the blocker.

    Unless there are any other ideas?

  8. luacmartins commented on Nov 28, 2023

    @luacmartins
    Contributor

    This is a new feature, so I think it'd be ok to demote that to NAB since it doesn't really break any functionality (the option just shouldn't be there) and we can work on an Auth PR to filter those cards out. What do you think?

  9. marcaaron commented on Nov 28, 2023

    @marcaaron
    Contributor

    It's a core flow so I'll raise a PR to hide the row for everyone temporarily. My preference would be to avoid shipping the broken code.

  10. added
    ReviewingHas a PR in review
    and removed on Nov 28, 2023
  11. 42 remaining items

  12. melvin-bot commented on Jan 1, 2024

    @melvin-bot

    @anmurali, @grgia, @alitoshmatov 10 days overdue. I'm getting more depressed than Marvin.

  13. melvin-bot commented on Jan 1, 2024

    @melvin-bot

    @anmurali, @grgia, @alitoshmatov 10 days overdue. I'm getting more depressed than Marvin.

  14. grgia commented on Jan 2, 2024

    @grgia
    Contributor

    @anmurali this one's ready for payment- let me know if you need anything else from me!

  15. grgia commented on Jan 5, 2024

    @grgia
    Contributor
  16. melvin-bot commented on Jan 9, 2024

    @melvin-bot

    @anmurali, @grgia, @alitoshmatov Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

  17. anmurali commented on Jan 10, 2024

    @anmurali

    Offer is https://www.upwork.com/nx/wm/offer/100368742
    Pls let me know if the amount is right.

  18. alitoshmatov commented on Jan 10, 2024

    @alitoshmatov
    Contributor

    Can't access upwork job

  19. anmurali commented on Jan 10, 2024

    @anmurali
  20. anmurali commented on Jan 11, 2024

    @anmurali

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

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions