Skip to content

[HOLD for payment 2023-02-10] [$1000] Android - Payment- White line appears when clicking on "Add Payment method" button #14563

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. Launch the app
  2. Log i with any account
  3. Tab Setting>Payment
  4. Tab button Add payment method

Expected Result:

No white line appears when clicking on "Add Payment method" button

Actual Result:

White line appears when clicking on "Add Payment method" button

Workaround:

Unknown

Platforms:

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

  • Android / native

Version Number: 1.2.59.1

Reproducible in staging?: Yes

Reproducible in production?: Yes

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

Bug5911544_android_flash_2501.mp4

Expensify/Expensify Issue URL:

Issue reported by: Applause - Internal Team

Slack conversation:

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~01dc6f5499efed1dd6
  • Upwork Job ID: 1619005122333753344
  • Last Price Increase: 2023-01-27

Activity

  1. added
    BugSomething is broken. Auto assigns a BugZero manager.
    on Jan 25, 2023
  2. locked and limited conversation to collaborators on Jan 25, 2023
  3. added
    ExternalAdded to denote the issue can be worked on by a contributor
    on Jan 27, 2023
  4. unlocked this conversation on Jan 27, 2023
  5. changed the title [-]Android - Payment- White line appears when clicking on "Add Payment method" button[/-] [+][$1000] Android - Payment- White line appears when clicking on "Add Payment method" button[/+] on Jan 27, 2023
  6. melvin-bot commented on Jan 27, 2023

    @melvin-bot
  7. melvin-bot commented on Jan 27, 2023

    @melvin-bot

    Current assignee @flaviadefaria is eligible for the External assigner, not assigning anyone new.

  8. flaviadefaria commented on Jan 27, 2023

    @flaviadefaria
    Contributor

    It's a subtle one but was able to reproduce it.

  9. melvin-bot commented on Jan 27, 2023

    @melvin-bot

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

  10. added
    Help WantedApply this label when an issue is open to proposals by contributors
    on Jan 27, 2023
  11. melvin-bot commented on Jan 27, 2023

    @melvin-bot

    Triggered auto assignment to @arosiclair (External), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

  12. 32 remaining items

  13. flaviadefaria commented on Feb 13, 2023

    @flaviadefaria
    Contributor

    @mollfpr @arosiclair @situchan please fill out the checklist above. Did the PR introduce a bug?

  14. arosiclair commented on Feb 13, 2023

    @arosiclair
    Contributor

    Since this is an upstream issue with react-native-modal, this may have never worked correctly but became more noticeable after our redesign. What do you think @mollfpr?

  15. mollfpr commented on Feb 13, 2023

    @mollfpr
    Contributor

    @arosiclair Yeah, I agree with you.

  16. mollfpr commented on Feb 14, 2023

    @mollfpr
    Contributor

    @situchan Could you propose the regression test for this issue? Thanks!

  17. situchan commented on Feb 14, 2023

    @situchan
    Contributor

    Actually, regression test will be same as in QA step.

    Regression Test Proposal

    • Login with any account
    • Go to Settings -> Payments
    • Tap "Add payment method" button
    • Verify that no white area shows on "Add Payment method" button while modal is opening
    • Do we agree 👍 or 👎
  18. mollfpr commented on Feb 14, 2023

    @mollfpr
    Contributor

    @arosiclair Is it possible to add the regression test if the issue is regarding the app's appearance?

  19. flaviadefaria commented on Feb 14, 2023

    @flaviadefaria
    Contributor

    @situchan was assigned on Jan 31st. PR Merged on Feb 2nd = 50% bonus for @situchan and @mollfpr.
    I'll pay you both now.

  20. flaviadefaria commented on Feb 14, 2023

    @flaviadefaria
    Contributor

    Alright, I have now paid @situchan and @mollfpr.
    Waiting on @arosiclair to see if we need to add a regression test.

  21. arosiclair commented on Feb 15, 2023

    @arosiclair
    Contributor

    IMO I don't think this regression test is needed, but I asked in slack here

  22. flaviadefaria commented on Feb 15, 2023

    @flaviadefaria
    Contributor

    Thanks @arosiclair! Just saw the Slack thread and since we don't need to add a regression test and everyone has been paid we're good to close this issue. Thanks, everyone!

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