Skip to content

Revert "Moving Per Diem expense from self DM" - #77651

Closed
cretadn22 wants to merge 1 commit into
Expensify:mainfrom
cretadn22:revert-76159-feat/72239-per-diem-field-create-report
Closed

cretadn22 wants to merge 1 commit into
Expensify:mainfrom
cretadn22:revert-76159-feat/72239-per-diem-field-create-report

Conversation

@cretadn22

Copy link
Copy Markdown
Contributor

Reverts #76159

@koko57

koko57 commented Dec 15, 2025

Copy link
Copy Markdown
Contributor

@cretadn22 why are you reverting the whole PR? I can fix it quickly

@cretadn22

Copy link
Copy Markdown
Contributor Author

@koko57 No, I only want to test the code before your change. It seems there’s no difference between the behavior before and after your PR.

@cretadn22 cretadn22 closed this Dec 15, 2025
@koko57

koko57 commented Dec 15, 2025

Copy link
Copy Markdown
Contributor

@cretadn22 ok, thanks! 😅

@codecov

codecov Bot commented Dec 15, 2025

Copy link
Copy Markdown

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/libs/PolicyUtils.ts 55.26% <ø> (+0.06%) ⬆️
...es/iou/request/step/IOURequestEditReportCommon.tsx 85.07% <0.00%> (+2.98%) ⬆️
...rc/pages/iou/request/step/IOURequestEditReport.tsx 0.00% <0.00%> (ø)
...rc/pages/iou/request/step/IOURequestStepReport.tsx 0.00% <0.00%> (ø)
...c/components/ReportActionItem/MoneyRequestView.tsx 0.26% <0.00%> (-0.25%) ⬇️
... and 7 files with indirect coverage changes

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.

2 participants