Skip to content

[Refactor] Able to remove PayPal account while offline, when enabled wifi - account still present #11260

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 in any account
  3. Go to Settings - Payments
  4. Add Paypal account
  5. Disable in internet connection
  6. Remove the Paypal account while offline
  7. Enable the internet connection

Expected Result:

Paypal account should not be visible after enable the internet connection

Actual Result:

Able to remove PayPal account while offline, when enabled wifi - account still present

Workaround:

Unknown

Platform:

Where is this issue occurring?

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

Version Number: 1.2.6.0

Reproducible in staging?: Yes

Reproducible in production?: n/a

Email or phone of affected tester (no customers): any

Logs: https://stackoverflow.com/c/expensify/questions/4856

Notes/Photos/Videos: Any additional supporting documentation

Screen_Recording_20220923-222858_New.Expensify.mp4

Expensify/Expensify Issue URL:

Issue reported by: Applause - Internal Team

Slack conversation:

View all open jobs on GitHub

Activity

  1. melvin-bot commented on Sep 24, 2022

    @melvin-bot

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

  2. Justicea83 commented on Sep 26, 2022

    @Justicea83
    Contributor

    I can't reproduce this, but it can be external

  3. added
    ExternalAdded to denote the issue can be worked on by a contributor
    on Sep 26, 2022
  4. removed their assignment
    on Sep 26, 2022
  5. melvin-bot commented on Sep 26, 2022

    @melvin-bot

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

  6. melvin-bot commented on Sep 26, 2022

    @melvin-bot

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

  7. added
    Help WantedApply this label when an issue is open to proposals by contributors
    on Sep 26, 2022
  8. melvin-bot commented on Sep 26, 2022

    @melvin-bot

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

  9. changed the title [-]Android - Able to remove PayPal account while offline, when enabled wifi - account still present[/-] [+][$250] Android - Able to remove PayPal account while offline, when enabled wifi - account still present[/+] on Sep 26, 2022
  10. added
    InternalRequires API changes or must be handled by Expensify staff
    and removed
    ExternalAdded to denote the issue can be worked on by a contributor
    on Sep 26, 2022
  11. added this to the milestone on Sep 26, 2022
  12. changed the title [-][$250] Android - Able to remove PayPal account while offline, when enabled wifi - account still present[/-] [+][Refactor] Able to remove PayPal account while offline, when enabled wifi - account still present[/+] on Sep 26, 2022
  13. mountiny commented on Sep 26, 2022

    @mountiny
    Contributor

    I can reproduce this and I think this must be Internal issue. We had refactored the Delete paypal.me address in this PR (cc @danieldoglas) but we need to make sure to check the Web part of API and see what is done there to make sure client is updated correctly (and the paypal.me address does not appear after being deleted.

  14. added
    ImprovementItem broken or needs improvement.
    and removed
    Help WantedApply this label when an issue is open to proposals by contributors
    on Sep 26, 2022
  15. removed their assignment
    on Sep 26, 2022
  16. mountiny commented on Sep 26, 2022

    @mountiny
    Contributor

    Asked in Slack for an internal employee to investigate this.

  17. self-assigned this
    on Sep 26, 2022
  18. arosiclair commented on Sep 27, 2022

    @arosiclair
    Contributor

    I can confirm this is an issue with API reads being executed before offline writes on reconnect. We're running ReconnectApp and OpenPaymentsPage before DeletePayPalMeAddress which causes the deleted paypal to be re-added before it is eventually removed again.

    Screen Shot 2022-09-27 at 4 51 01 PM

    Closing this in favor of https://github.com/Expensify/Expensify/issues/229696 which should fix the issue

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

Metadata

Metadata

Assignees

Labels

DailyKSv2EngineeringImprovementItem broken or needs improvement.InternalRequires API changes or must be handled by Expensify staff

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions