Skip to content

[CP Staging] Revert "Remove Onyx.connect() for the key: ONYXKEYS.BETAS in src/libs/ReportUtils.ts."#80238

Merged
Valforte merged 1 commit intomainfrom
revert-78853-krishna2323/issue/66417
Jan 22, 2026
Merged

[CP Staging] Revert "Remove Onyx.connect() for the key: ONYXKEYS.BETAS in src/libs/ReportUtils.ts."#80238
Valforte merged 1 commit intomainfrom
revert-78853-krishna2323/issue/66417

Conversation

@Valforte
Copy link
Copy Markdown
Contributor

Reverts #78853

@Valforte Valforte requested review from a team as code owners January 22, 2026 14:25
@melvin-bot melvin-bot bot requested review from trjExpensify and youssef-lr and removed request for a team January 22, 2026 14:25
@melvin-bot
Copy link
Copy Markdown

melvin-bot bot commented Jan 22, 2026

@youssef-lr Please copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button]

@codecov
Copy link
Copy Markdown

codecov bot commented Jan 22, 2026

Codecov Report

❌ Looks like you've decreased code coverage for some files. Please write tests to increase, or at least maintain, the existing level of code coverage. See our documentation here for how to interpret this table.

Files with missing lines Coverage Δ
src/components/AddUnreportedExpenseFooter.tsx 0.00% <ø> (ø)
src/components/MoneyRequestHeader.tsx 0.00% <ø> (ø)
src/components/ProcessMoneyReportHoldMenu.tsx 0.00% <ø> (ø)
src/components/ReportWelcomeText.tsx 88.88% <ø> (-0.31%) ⬇️
src/hooks/useDeleteTransactions.ts 20.27% <ø> (-1.07%) ⬇️
src/libs/QuickActionUtils.ts 47.19% <100.00%> (ø)
src/libs/ReportSecondaryActionUtils.ts 92.00% <100.00%> (ø)
src/libs/actions/IOU/Duplicate.ts 85.31% <ø> (ø)
src/libs/actions/IOU/MoneyRequest.ts 100.00% <ø> (ø)
src/libs/actions/Report.ts 53.74% <100.00%> (ø)
... and 30 more
... and 7 files with indirect coverage changes

@Valforte Valforte merged commit 03cf165 into main Jan 22, 2026
38 of 45 checks passed
@Valforte Valforte deleted the revert-78853-krishna2323/issue/66417 branch January 22, 2026 14:49
@melvin-bot melvin-bot bot added the Emergency label Jan 22, 2026
@melvin-bot
Copy link
Copy Markdown

melvin-bot bot commented Jan 22, 2026

@Valforte looks like this was merged without a test passing. Please add a note explaining why this was done and remove the Emergency label if this is not an emergency.

@Valforte
Copy link
Copy Markdown
Contributor Author

Not Emergency, straight revert

@OSBotify
Copy link
Copy Markdown
Contributor

✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

@Beamanator Beamanator changed the title Revert "Remove Onyx.connect() for the key: ONYXKEYS.BETAS in src/libs/ReportUtils.ts." [CP Staging] Revert "Remove Onyx.connect() for the key: ONYXKEYS.BETAS in src/libs/ReportUtils.ts." Jan 22, 2026
OSBotify pushed a commit that referenced this pull request Jan 22, 2026
…ue/66417

Revert "Remove Onyx.connect() for the key: ONYXKEYS.BETAS in src/libs/ReportUtils.ts."

(cherry picked from commit 03cf165)

(cherry-picked to staging by Beamanator)
@OSBotify OSBotify added the CP Staging marks PRs that have been CP'd to staging label Jan 22, 2026
@OSBotify
Copy link
Copy Markdown
Contributor

🚀 Cherry-picked to staging by https://github.com/Beamanator in version: 9.3.6-3 🚀

platform result
🕸 web 🕸 success ✅
🤖 android 🤖 success ✅
🍎 iOS 🍎 success ✅

@OSBotify
Copy link
Copy Markdown
Contributor

🚀 Deployed to production by https://github.com/Beamanator in version: 9.3.6-4 🚀

platform result
🕸 web 🕸 success ✅
🤖 android 🤖 success ✅
🍎 iOS 🍎 success ✅

@OSBotify
Copy link
Copy Markdown
Contributor

🚀 Cherry-picked to staging by https://github.com/Beamanator in version: 9.3.7-0 🚀

platform result
🕸 web 🕸 success ✅
🤖 android 🤖 success ✅
🍎 iOS 🍎 success ✅

@OSBotify
Copy link
Copy Markdown
Contributor

🚀 Deployed to production by https://github.com/Beamanator in version: 9.3.7-3 🚀

platform result
🕸 web 🕸 success ✅
🤖 android 🤖 success ✅
🍎 iOS 🍎 failure ❌

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CP Staging marks PRs that have been CP'd to staging

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants