[Due for Payment 2025-10-21] Remove `Onyx.connect()` for the key: `ONYXKEYS.NVP_ACTIVE_POLICY_ID` in `src/libs/actions/IOU.ts`

[tgolen]

This is part of deprecating Onyx.connect. Parent Issue

Details

• Module: src/libs/actions/IOU.ts
• Onyx Key: ONYXKEYS.NVP_ACTIVE_POLICY_ID
• Reference: Line 837: ONYXKEYS.NVP_ACTIVE_POLICY_ID

Be sure to refactor methods using test driven development:

1. Create a unit test for methods before refactoring
2. Refactor the methods
3. Ensure the unit test still runs after refactoring
4. Reinforce the unit testing with a functional test and a QA test
5. Once you are done, decrease the number of allowable ESLint warnings here to equal the number of Onyx.connect references remaining in the code

Issue Owner

Current Issue Owner: @bfitzexpensify

[tgolen]

Warning

DO NOT POST PROPOSALS ON THIS ISSUE!!

Bug-Zero Instructions

• This issue is CLOSED for proposals
• It is a special issue and part of a bigger project
• A contributor will be manually assigned to work on this issue and they do not need to submit a proposal
• The bug-zero person only needs to take care of payments to the contributor for the PR and the C+ for reviews
• There do not need to be any regression tests added or updated since this is only refactoring and shouldn't change any existing functionality

[melvin-bot]

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

[melvin-bot]

@danieldoglas @bfitzexpensify this issue was created 2 weeks ago. Are we close to a solution? Let's make sure we're treating this as a top priority. Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

[danieldoglas]

Not overdue, will be worked soon.