Repository navigation
[Partial Payment] Remove Onyx.connect() for the key: ONYXKEYS.CREDENTIALS in src/libs/actions/Session/index.ts #67785
Description
This is part of deprecating Onyx.connect. Parent Issue
Details
- Module: src/libs/actions/Session/index.ts
- Onyx Key:
ONYXKEYS.CREDENTIALS - Reference:
Line 104: ONYXKEYS.CREDENTIALS
Be sure to refactor methods using test driven development:
- Create a unit test for methods before refactoring
- Refactor the methods
- Ensure the unit test still runs after refactoring
- Reinforce the unit testing with a functional test and a QA test
- Once you are done, decrease the number of allowable ESLint warnings here to equal the number of
Onyx.connectreferences remaining in the code
Activity
danieldoglas commentedon Aug 4, 2025
Warning
DO NOT POST PROPOSALS ON THIS ISSUE!!
Bug-Zero Instructions
src/libs/actions/Session/index.tsto removeOnyx.connect()references #67781melvin-bot commentedon Aug 4, 2025
Triggered auto assignment to @sakluger (
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.sakluger commentedon Sep 6, 2025
@danieldoglas what's the status of this one?
91 remaining items
mallenexpensify commentedon Oct 27, 2026
Payment Summary
Contributor+: @huult due $250 via NewDot
Contributor+: @shubham1206agra due $500 via NewDot
Contributor+: @cretadn22 due $750 via NewDot
Is that correct?!?! Tag me if not, cc @danieldoglas
Updated title to reflect partial payments, leaving issue open
[-][Due for payment 2026-10-26] Remove `Onyx.connect()` for the key: `ONYXKEYS.CREDENTIALS` in `src/libs/actions/Session/index.ts`[/-][+][Partial Payment] Remove `Onyx.connect()` for the key: `ONYXKEYS.CREDENTIALS` in `src/libs/actions/Session/index.ts`[/+]huult commentedon Oct 29, 2026
I think we've summarized one part for the payment ticket, and since it's on hold, we can move this ticket to weekly.
huult commentedon Nov 11, 2026
Not overdue