Skip to content

[HOLD for payment 2024-07-26] [$250] Send invoice - "Choose a payment method" disappears after returning from Pay elsewhere page #44830

Closed

Description

@izarutskaya

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.4-0
Reproducible in staging?: Y
Reproducible in production?: Y
Found when validating PR : #43036
Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team

Action Performed:

  1. Go to staging.new.expensify.com
  2. [User A] Send an invoice to user B.
  3. [User B] Go to invoice chat.
  4. [User B] Click Pay button.
  5. [User B] Click Pay as individual or Pay as business.
  6. [User B] Click on the back button.

Expected Result:

"Choose a payment method below" in the payment method selection page will persist.

Actual Result:

"Choose a payment method below" in the payment method selection page disappears.

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

Bug6532611_1720055363822.20240704_090314.1.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~01ace2c2d5df911f17
  • Upwork Job ID: 1809200291724480759
  • Last Price Increase: 2024-07-05
  • Automatic offers:
    • cretadn22 | Contributor | 103046339
Issue OwnerCurrent Issue Owner: @sonialiap

Activity

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

    @melvin-bot

    Triggered auto assignment to @sonialiap (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. izarutskaya commented on Jul 4, 2024

    @izarutskaya
    Author

    We think this issue might be related to the #vip-bills.

  4. daledah commented on Jul 4, 2024

    @daledah
    Contributor

    Proposal

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

    "Choose a payment method below" in the payment method selection page disappears.

    What is the root cause of that problem?

    We don't use setState when click back button in

    enteredSubMenuIndexes.splice(-1);

    so state enteredSubMenuIndexes is not updated and popover isn't rendered again

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

    We should create a function renderHeader() and it'll update header everytime enteredSubMenuIndexes changes

        const renderHeader = () => {
            if(!headerText || enteredSubMenuIndexes.length > 0) {
                return;
            }
            return <Text style={[styles.createMenuHeaderText, styles.ph5, styles.pv3]}>{headerText}</Text>
        }
    
    

    What alternative solutions did you explore? (Optional)

    Using setState in this case

    setEnteredSubMenuIndexes((prevState) => prevState.splice(-1)) 
    
  5. cretadn22 commented on Jul 4, 2024

    @cretadn22
    Contributor

    Proposal

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

    "Choose a payment method" disappears after returning from Pay elsewhere page

    What is the root cause of that problem?

    We update enteredSubMenuIndexes state directly

    enteredSubMenuIndexes.splice(-1);

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

    Need to utilize setState to adjust the state value

    enteredSubMenuIndexes.splice(-1);

    setEnteredSubMenuIndexes((prevState) => prevState.slice(0, -1)) 
    

    What alternative solutions did you explore? (Optional)

  6. added
    ExternalAdded to denote the issue can be worked on by a contributor
    on Jul 5, 2024
  7. melvin-bot commented on Jul 5, 2024

    @melvin-bot
  8. changed the title [-] Send invoice - "Choose a payment method" disappears after returning from Pay elsewhere page[/-] [+][$250] Send invoice - "Choose a payment method" disappears after returning from Pay elsewhere page[/+] on Jul 5, 2024
  9. added
    Help WantedApply this label when an issue is open to proposals by contributors
    on Jul 5, 2024
  10. melvin-bot commented on Jul 5, 2024

    @melvin-bot

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

  11. Abdelrahman286 commented on Jul 6, 2024

    @Abdelrahman286

    it would be better if you explain with voice

  12. 38 remaining items

  13. JmillsExpensify commented on Jul 29, 2024

    @JmillsExpensify
    Contributor

    @sonialiap please add a payment summary when you get a chance.

  14. sonialiap commented on Jul 30, 2024

    @sonialiap
    Contributor

    Since it seems like there was extra work put into this, I'm increasing the base price to $500

    Payment summary:

    • @daledah $250 for the proposal - offer sent in Upwork - paid in Upwork ✔️
    • @cretadn22 $500 for the fix - paid in Upwork ✔️
    • @mollfpr $500 for the c+ - please request in ND

    Does this sound fair to everyone?

  15. JmillsExpensify commented on Jul 30, 2024

    @JmillsExpensify
    Contributor

    $500 approved for @mollfpr

  16. cretadn22 commented on Aug 1, 2024

    @cretadn22
    Contributor

    @JmillsExpensify @sonialiap Could you please approve my offer?

  17. daledah commented on Aug 2, 2024

    @daledah
    Contributor

    @sonialiap Sounds good to me thanks, could you send the offer to my UW profile here? https://www.upwork.com/freelancers/~0138d999529f34d33f

  18. sonialiap commented on Aug 2, 2024

    @sonialiap
    Contributor

    @daledah thanks for the upwork link! Offer sent 💃

  19. daledah commented on Aug 3, 2024

    @daledah
    Contributor

    @sonialiap Accepted the offer thx

  20. sonialiap commented on Aug 5, 2024

    @sonialiap
    Contributor

    Everyone has been 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

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions