Skip to content

[$250] [HOLD for payment 2024-07-10] [HybridApp] Do not allow logout in NewDot HybridApp #43336

Closed

Description

@AndrewGable

Problem

If a user logs out of HybridApp via NewDot, many weird things can happen including crashes.

Solution

Do not allow HybridApp users to logout via NewDot

cc @staszekscp

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~01970b1b535b50d220
  • Upwork Job ID: 1810704730661899086
  • Last Price Increase: 2024-07-09
Issue OwnerCurrent Issue Owner: @

Activity

  1. self-assigned this
    on Jun 8, 2024
  2. moved this to Release 2: Summer 2024 (Aug) in [#whatsnext] #wave-collecton Jun 10, 2024
  3. AndrewGable commented on Jun 10, 2024

    @AndrewGable
    ContributorAuthor

    Hopefully will get to this tomorrow

  4. AndrewGable commented on Jun 11, 2024

    @AndrewGable
    ContributorAuthor

    Ok so I think we need to actually expand the scope of this one a bit:

    1. Do not show the logout button on NewDot
    2. Anytime we are going to log out of NewDot, we should go back to OldApp and then log the user out there (if we have to)

    The reason is we cannot support the NewDot sign in page for HybridApp, since OldApp contains all the authentication logic for HybridApp.

  5. AndrewGable commented on Jun 12, 2024

    @AndrewGable
    ContributorAuthor

    @shawnborton is still having issues with an automatic log out:

    RPReplay_Final1718212810.MP4

    We should look for logs to determine why he is being kicked out.

  6. melvin-bot commented on Jun 17, 2024

    @melvin-bot

    @AndrewGable Eep! 4 days overdue now. Issues have feelings too...

  7. AndrewGable commented on Jun 18, 2024

    @AndrewGable
    ContributorAuthor

    Going to look into this one soon

  8. AndrewGable commented on Jun 19, 2024

    @AndrewGable
    ContributorAuthor

    cc @WoLewicki - I was thinking this could be a good issue to look into.

    Basically we do not want to ever go to the NewDot login page on HybridApp.

    1. We want to hide the Sign out button if we are on HybridApp
    2. When the user should be signed out automatically (due to an authentication error), we want to exit NewDot instead of showing them the login page on NewDot.
  9. 21 remaining items

  10. added
    ExternalAdded to denote the issue can be worked on by a contributor
    on Jul 9, 2024
  11. melvin-bot commented on Jul 9, 2024

    @melvin-bot

    Triggered auto assignment to @abekkala (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.

  12. changed the title [-][HOLD for payment 2024-07-10] [HybridApp] Do not allow logout in NewDot HybridApp[/-] [+][$250] [HOLD for payment 2024-07-10] [HybridApp] Do not allow logout in NewDot HybridApp[/+] on Jul 9, 2024
  13. melvin-bot commented on Jul 9, 2024

    @melvin-bot
  14. added
    Help WantedApply this label when an issue is open to proposals by contributors
    and removed on Jul 9, 2024
  15. melvin-bot commented on Jul 9, 2024

    @melvin-bot

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

  16. removed
    Help WantedApply this label when an issue is open to proposals by contributors
    on Jul 9, 2024
  17. mallenexpensify commented on Jul 9, 2024

    @mallenexpensify
    Contributor

    @akinwale can you please accept the job and reply here once you have?
    https://www.upwork.com/jobs/~01970b1b535b50d220

    @abekkala , please pay tomorrow, PR hit production July 3. Thx

  18. akinwale commented on Jul 9, 2024

    @akinwale
    Contributor

    @mallenexpensify I did not receive an offer for the job. Do I just go ahead to submit a proposal on Upwork?

  19. mallenexpensify commented on Jul 10, 2024

    @mallenexpensify
    Contributor

    Strange, I probably messed something up. You should be able to accept the job now.
    https://www.upwork.com/jobs/~01970b1b535b50d220

  20. abekkala commented on Jul 10, 2024

    @abekkala
    Contributor

    PAYMENT SUMMARY FOR JULY 10

    • Fix: @WoLewicki - SWM, no payment needed
    • PR Review: [$250] payment sent and contract ended - thank you! 🎉
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