Skip to content

Revert "fix Home page Begin button behavior"#83707

Merged
justinpersaud merged 1 commit intomainfrom
revert-83019-fix/82102
Feb 27, 2026
Merged

Revert "fix Home page Begin button behavior"#83707
justinpersaud merged 1 commit intomainfrom
revert-83019-fix/82102

Conversation

@justinpersaud
Copy link
Contributor

Reverts #83019

@justinpersaud justinpersaud self-assigned this Feb 27, 2026
@justinpersaud justinpersaud requested review from a team as code owners February 27, 2026 15:44
@melvin-bot melvin-bot bot requested review from joekaufmanexpensify and removed request for a team February 27, 2026 15:44
@melvin-bot melvin-bot bot requested a review from Gonals February 27, 2026 15:44
@melvin-bot
Copy link

melvin-bot bot commented Feb 27, 2026

@Gonals 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]

@melvin-bot melvin-bot bot removed the request for review from a team February 27, 2026 15:44
@justinpersaud justinpersaud merged commit a7fc7c3 into main Feb 27, 2026
30 of 32 checks passed
@justinpersaud justinpersaud deleted the revert-83019-fix/82102 branch February 27, 2026 15:47
@melvin-bot melvin-bot bot added the Emergency label Feb 27, 2026
@melvin-bot
Copy link

melvin-bot bot commented Feb 27, 2026

@justinpersaud 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.

@justinpersaud
Copy link
Contributor Author

Straight revert

@github-actions
Copy link
Contributor

🚧 @justinpersaud has triggered a test Expensify/App build. You can view the workflow run here.

@codecov
Copy link

codecov bot commented Feb 27, 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/pages/home/ForYouSection/index.tsx 88.88% <ø> (-11.12%) ⬇️
src/selectors/Todos.ts 100.00% <ø> (ø)
... and 14 files with indirect coverage changes

@OSBotify
Copy link
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.

@OSBotify
Copy link
Contributor

OSBotify commented Mar 2, 2026

🚀 Deployed to staging by https://github.com/justinpersaud in version: 9.3.28-0 🚀

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

@m-natarajan
Copy link

@justinpersaud @NikkiWines No QA steps, OK to checkoff this?

@NikkiWines
Copy link
Contributor

Yes, it's a revert of another PR @m-natarajan

@OSBotify
Copy link
Contributor

OSBotify commented Mar 4, 2026

🚀 Deployed to staging by https://github.com/justinpersaud in version: 9.3.30-0 🚀

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

@OSBotify
Copy link
Contributor

OSBotify commented Mar 4, 2026

🚀 Deployed to production by https://github.com/blimpich in version: 9.3.30-3 🚀

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

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants