[CP Staging] Revert "Make AddUnreportedExpense use new SelectionList" - #76275
Conversation
mountiny
changed the title
mountiny
left a comment
mountiny
left a comment
There was a problem hiding this comment.
Confirmed the revert changes are right
|
@mountiny looks like this was merged without a test passing. Please add a note explaining why this was done and remove the |
|
Straight revert, not an emergency |
Codecov Report✅ Changes either increased or maintained existing code coverage, great job!
|
|
✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release. |
…t-migrate-AddUnreportedExpense [CP Staging] Revert "Make AddUnreportedExpense use new SelectionList" (cherry picked from commit a94d3c3) (cherry-picked to staging by mountiny)
…765400767-1 🍒 Cherry pick PR #76275 to staging 🍒
|
🚀 Cherry-picked to staging by https://github.com/mountiny in version: 9.2.65-5 🚀
|
|
🚀 Deployed to production by https://github.com/marcaaron in version: 9.2.65-6 🚀
|
|
🚀 Cherry-picked to staging by https://github.com/mountiny in version: 9.2.66-0 🚀
|
|
🚀 Deployed to production by https://github.com/jasperhuangg in version: 9.2.66-3 🚀
|
3 participants
Reverts #75891
The reverted PR added 2 deploy blockers and is currently undergoing consultation regarding the design changes that were introduced.
Deploy blockers:
[$250] Android - Reports - Blank space displayed above expenses in the list from the Add unreported expense #76233
Report - The first item is highlighted when searching and selecting an expense #76207