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
[FINERACT-1678] Inline COB API #2675
Conversation
7f66876
to
5ee624b
Compare
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.
Overall I'm still confused by the naming in the PR. The inline functionality is not tied to COBs but any partitioned job in the future. Let's get rid of this naming and be more generic.
Plus, I'd love to see at least a single integration test for the inline COB's positive scenario.
fineract-provider/src/main/java/org/apache/fineract/cob/api/InlineCOBResource.java
Outdated
Show resolved
Hide resolved
fineract-provider/src/main/java/org/apache/fineract/cob/api/InlineCOBResource.java
Outdated
Show resolved
Hide resolved
fineract-provider/src/main/java/org/apache/fineract/cob/api/InlineCOBResource.java
Outdated
Show resolved
Hide resolved
...ovider/src/main/java/org/apache/fineract/cob/loan/InlineCOBBuildExecutionContextTasklet.java
Outdated
Show resolved
Hide resolved
...ovider/src/main/java/org/apache/fineract/cob/loan/InlineCOBBuildExecutionContextTasklet.java
Outdated
Show resolved
Hide resolved
fineract-provider/src/main/java/org/apache/fineract/cob/service/InlineCOBType.java
Outdated
Show resolved
Hide resolved
...provider/src/main/java/org/apache/fineract/cob/service/InlineLoanCOBExecutorServiceImpl.java
Outdated
Show resolved
Hide resolved
...provider/src/main/java/org/apache/fineract/cob/service/InlineLoanCOBExecutorServiceImpl.java
Outdated
Show resolved
Hide resolved
...provider/src/main/java/org/apache/fineract/cob/service/InlineLoanCOBExecutorServiceImpl.java
Outdated
Show resolved
Hide resolved
...provider/src/main/java/org/apache/fineract/cob/service/InlineLoanCOBExecutorServiceImpl.java
Outdated
Show resolved
Hide resolved
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.
I'm certain we can improve test coverage
3a2483c
to
4cd30c6
Compare
fineract-provider/src/main/java/org/apache/fineract/cob/loan/InlineCOBLoanItemProcessor.java
Outdated
Show resolved
Hide resolved
...er/src/main/java/org/apache/fineract/cob/loan/InlineLoanCOBBuildExecutionContextTasklet.java
Outdated
Show resolved
Hide resolved
fineract-provider/src/main/java/org/apache/fineract/cob/loan/LoanItemProcessor.java
Show resolved
Hide resolved
fineract-provider/src/main/java/org/apache/fineract/cob/loan/LoanItemProcessor.java
Outdated
Show resolved
Hide resolved
fineract-provider/src/main/java/org/apache/fineract/cob/loan/InlineCOBLoanItemReader.java
Show resolved
Hide resolved
fineract-provider/src/main/java/org/apache/fineract/cob/loan/LoanItemReader.java
Outdated
Show resolved
Hide resolved
|
||
@Override | ||
public CommandProcessingResult processCommand(JsonCommand command) { | ||
List<String> split = Splitter.on("/").splitToList(command.getUrl()); |
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.
Still don't agree with this, please fix it.
...act-provider/src/main/java/org/apache/fineract/cob/service/InlineCOBExecutionDataParser.java
Outdated
Show resolved
Hide resolved
...provider/src/main/java/org/apache/fineract/cob/service/InlineLoanCOBExecutorServiceImpl.java
Show resolved
Hide resolved
...provider/src/main/java/org/apache/fineract/cob/service/InlineLoanCOBExecutorServiceImpl.java
Outdated
Show resolved
Hide resolved
7ee1e2f
to
9d08a5f
Compare
9d08a5f
to
5b6ebd0
Compare
Description
This PR introduces the Inline Loan COB job.
A new API has been introduced to execute Inline COBs.