[Due for payment 2026-10-15] Remove Onyx.connect() for the key: ONYXKEYS.COLLECTION.REPORT_DRAFT in src/libs/ReportUtils.ts

@mallenexpensify

This is part of deprecating Onyx.connect. Parent Issue

Details

Module: src/libs/ReportUtils.ts
Onyx Key: ONYXKEYS.COLLECTION.REPORT_DRAFT
Reference: Line 931: ONYXKEYS.COLLECTION.REPORT_DRAFT

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.connect references remaining in the code

Issue Owner

Current Issue Owner: @mallenexpensify

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

@bfitzexpensify

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.

@tgolen

@tgolen @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!

This is a long-running project, so this will be gotten to eventually.

@mallenexpensify

Hi @mallenexpensify since this issue requires too many PRs, I'd like to request the payment for the next batch of PRs that were already deployed to production. We still need to keep this issue open for more upcoming PRs.

#89939 -> 250
#91279 -> 250
#92964 -> 250
#98960 -> 250
#100697 -> 250
#102400 -> 250

The total amount is $250 * 6 = 1500

@DylanDylann @tgolen Can you please confirm?

Agreed with that!

Total Approved: $1,500

@mallenexpensify

Triggered auto assignment to @mallenexpensify (Awaiting Payment)

@DylanDylann

Payment Summary

Contributor+: @DylanDylann due $1500 via NewDot
Contributor+: @daledah due $1500 via NewDot

Reopening this issue since we still have a few PRs to complete

Navigation Menu

Repository navigation

[Due for payment 2026-10-15] Remove `Onyx.connect()` for the key: `ONYXKEYS.COLLECTION.REPORT_DRAFT` in `src/libs/ReportUtils.ts` #66414

Details

Bug-Zero Instructions

131 remaining items

Total Approved: $1,500

Payment Summary

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

[Due for payment 2026-10-15] Remove Onyx.connect() for the key: ONYXKEYS.COLLECTION.REPORT_DRAFT in src/libs/ReportUtils.ts #66414

Description

Details

Activity

tgolen commented on Jul 16, 2025

Bug-Zero Instructions

melvin-bot commented on Jul 16, 2025

melvin-bot commented on Jul 30, 2025

tgolen commented on Aug 19, 2025

131 remaining items

daledah commented on Oct 15, 2026

tgolen commented on Oct 15, 2026

Total Approved: $1,500

melvin-bot commented on Oct 15, 2026

mallenexpensify commented on Oct 16, 2026

Payment Summary

daledah commented on Oct 20, 2026

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions

[Due for payment 2026-10-15] Remove `Onyx.connect()` for the key: `ONYXKEYS.COLLECTION.REPORT_DRAFT` in `src/libs/ReportUtils.ts` #66414