[$250] [Due for payment 2025-08-22] Remove `Onyx.connect()` for the key: `ONYXKEYS.COLLECTION.REPORT` in `src/libs/DistanceRequestUtils.ts`

[tgolen]

This is part of deprecating Onyx.connect. Parent Issue

Details

• Module: src/libs/DistanceRequestUtils.ts
• Onyx Key: ONYXKEYS.COLLECTION.REPORT
• Reference: Line 33: ONYXKEYS.COLLECTION.REPORT

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: @

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~021958820922126434619
• Upwork Job ID: 1958820922126434619
• Last Price Increase: 2025-08-22

[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 @laurenreidexpensify (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 @laurenreidexpensify 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!