Skip to content

Don't show Transfer balance option when current balance of Wallet = $0 #7601

Closed

Description

@kevinksullivan

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. Login to account with a Wallet that has a current / available balance of $0
  2. Go to Settings > Payments
  3. Verify balance shown in UI is $0

Expected Result:

There should be no option to transfer balance, since there is nothing to transfer out of the Wallet.

Actual Result:

The Transfer Balance option is shown, and on the next screen it becomes apparent that the user cannot transfer their balance, since the button is not active.

Workaround:

N/A

Platform:

  • Web
  • iOS
  • Android
  • Desktop App
  • Mobile Web

Reproducible in staging?: Yes
Reproducible in production?: Yes

Notes/Photos/Videos:
image
image

Slack conversation: https://expensify.slack.com/archives/C02NZ2HGJAZ/p1643834892220039?thread_ts=1643638216.390969&cid=C02NZ2HGJAZ

View all open jobs on GitHub

Activity

  1. MelvinBot commented on Feb 7, 2022

    @MelvinBot
    Contributor

    Triggered auto assignment to @flaviadefaria (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

  2. added and removed
    AutoAssignerTriageAuto assign issues for triage to an available triage team member
    on Feb 7, 2022
  3. MelvinBot commented on Feb 11, 2022

    @MelvinBot
    Contributor

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

  4. removed their assignment
    on Feb 11, 2022
  5. MelvinBot commented on Feb 21, 2022

    @MelvinBot
    Contributor

    @nickmurray47 Huh... This is 4 days overdue. Who can take care of this?

  6. MelvinBot commented on Feb 23, 2022

    @MelvinBot
    Contributor

    @nickmurray47 6 days overdue. This is scarier than being forced to listen to Vogon poetry!

  7. nickmurray47 commented on Feb 25, 2022

    @nickmurray47
    Contributor

    this has been merged and is just waiting deployment Melvin

  8. nickmurray47 commented on Mar 1, 2022

    @nickmurray47
    Contributor

    deployed to Prod, closing

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

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions