We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Coming from this conversation, there are some incorrect next steps showing on reports. We need to fix that.
The two updates below should be made to both NewDot and OldDot:
Waiting for <user> to add expense(s).
Waiting for <user> to submit expense(s).
Here's an example of where the next steps are incorrect in NewDot currently:
The text was updated successfully, but these errors were encountered:
Triggered auto assignment to @CortneyOfstad (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.
Bug
Sorry, something went wrong.
🚨 Edited by proposal-police: This proposal was edited at 2025-01-20 05:50:49 UTC.
Feature update
For this case:
App/src/libs/NextStepUtils.ts
Lines 112 to 114 in 177dd5e
The empty report has been handled by default.
empty report
Add a condition for open report with scheduled/delayed submit set to manual:
open report with scheduled/delayed submit set to manual
!harvesting?.enabled && autoReportingFrequency === CONST.POLICY.AUTO_REPORTING_FREQUENCIES.IMMEDIATE
And modify optimisticNextStep.message to Waiting for <user> to submit expense(s).
optimisticNextStep.message
NA
Update the NextStepUtilsTest for the default (the (1)) case here and the manual submit case here
NextStepUtilsTest
manual
CortneyOfstad
No branches or pull requests
Background
Coming from this conversation, there are some incorrect next steps showing on reports. We need to fix that.
Details
The two updates below should be made to both NewDot and OldDot:
Waiting for <user> to add expense(s).
Waiting for <user> to submit expense(s).
Screenshot
Here's an example of where the next steps are incorrect in NewDot currently:
The text was updated successfully, but these errors were encountered: