Skip to content

[HOLD #37281][$1000] Onyx data does not get removed cleanly on sign out #34473

Closed

Description

@m-natarajan

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:
Reproducible in staging?: needs reproduction ( Don't have account with several hundreds of chats)
Reproducible in production?:
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: @marcaaron
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1705012041676559

Action Performed:

  1. Signed into an account with many many chats
  2. Signed out
  3. Signed in with a brand new account (which should have only the Concierge chat)

Expected Result:

Nothing from the account that I signed out from should be persisted in the client data whatsoever.

Actual Result:

Tons of chats from the first account show up in the second account. A ton of Onyx keys still have values of {hasDraft: false} and there are also straight up reports that shouldn't be there at all. These also show up in the LHN.

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
2024-01-11_12-21-12
2024-01-11_12-22-20

2024-01-11_12-19-23

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~01211aa8ecc46d7a5e
  • Upwork Job ID: 1759703332875366400
  • Last Price Increase: 2024-03-24

Activity

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

    @melvin-bot

    Triggered auto assignment to @anmurali (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

  3. melvin-bot commented on Jan 16, 2024

    @melvin-bot

    @anmurali Whoops! This issue is 2 days overdue. Let's get this updated quick!

  4. anmurali commented on Jan 16, 2024

    @anmurali

    I cannot reproduce
    image
    The two reports I see are Concierge DM and domain room, and the new account has both.

    @marcaaron can you test this again and reopen if you still run into it?

  5. marcaaron commented on Jan 17, 2024

    @marcaaron
    Contributor

    It seems dependent on how much data you have in the first account when signing out. If you have a lot - not all of it gets cleared.

    We can close it - but feels like there is some obvious reason why this is happening and it should be patched at some point. Switching from one account to another is not such a common thing though so we can deprioritize it for now (but it is still happening for me).

    2024-01-16_14-41-03

  6. added
    InternalRequires API changes or must be handled by Expensify staff
    on Feb 19, 2024
  7. melvin-bot commented on Feb 19, 2024

    @melvin-bot
  8. melvin-bot commented on Feb 19, 2024

    @melvin-bot

    Triggered auto assignment to Contributor Plus for review of internal employee PR - @thesahindia (Internal)

  9. 87 remaining items

  10. added and removed
    ExternalAdded to denote the issue can be worked on by a contributor
    Help WantedApply this label when an issue is open to proposals by contributors
    on Mar 26, 2024
  11. mallenexpensify commented on Mar 26, 2024

    @mallenexpensify
    Contributor

    Put on hold, removed External and Help Wanted, bumped to Weekly

    Thanks @shubham1206agra for the tip!

  12. s77rt commented on Apr 5, 2024

    @s77rt
    Member

    #37281 is merged. Issue not longer reproducible. Let's close this

  13. added
    retest-weeklyApply this label if you want this issue tested on a Weekly basis by Applause
    on Apr 7, 2024
  14. mallenexpensify commented on Apr 7, 2024

    @mallenexpensify
    Contributor

    Throwing retest-weekly on this to confirm, then we can close. thx!

  15. anmurali commented on Apr 17, 2024

    @anmurali

    @mallenexpensify any updates?

  16. mallenexpensify commented on Apr 18, 2024

    @mallenexpensify
    Contributor

    Closing, just tried to repro with no luck
    image

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

Metadata

Metadata

Assignees

Labels

BugSomething is broken. Auto assigns a BugZero manager.WeeklyKSv2retest-weeklyApply this label if you want this issue tested on a Weekly basis by Applause

Type

No type

Projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions