Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
New issue
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
[WAVE7] Hold Requests: Approving/Paying expense reports with held requests #33124
[WAVE7] Hold Requests: Approving/Paying expense reports with held requests #33124
Changes from 60 commits
3f4df08
faebe01
f4297cd
317f5fc
da7e0a6
d38c0f6
818a841
ed3fe5e
275e17b
7301183
f28bd09
e237009
4e36be7
38f28b5
7fefad2
aa63896
cc5b1a0
adea766
7067e21
eb74222
19aa76d
32e7c7e
47c7c8d
c56681c
2fee6b7
7033f38
04636a8
b81c5df
20f568c
34e2d8c
d31c2a8
9eda512
beb1dc8
01e1bdb
7bd074d
3e90c40
2574741
75741ae
3c1edf5
14a7b65
c080092
9c4c695
914443e
d9a8fec
4734744
7428752
68aa153
46e626b
5c44e41
37311c3
69839f1
5c535dd
a8a414f
3cd3849
7f691ff
071ca7b
d35f1cb
a86501a
54d293b
9780889
cbb1a7b
6f99630
e538272
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@BartoszGrajdek Is there a reason that the
canAllowSettlement
condition is required here?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
true
value should have been passed since this function is handling full payment. ref: #39357There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
NAB: just making a note for later cleanup- we can probably move these string constants to https://github.com/Expensify/App/blob/main/src/CONST.ts