[
https://issues.apache.org/jira/browse/FINERACT-2477?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18061671#comment-18061671
]
saifulhuq commented on FINERACT-2477:
-------------------------------------
[~amanmittal003] Thanks for catching the overlap and linking the dependency.
I have pushed the final logic updates and resolved the local test conflicts on
FINERACT-2471 (PR #5465). It is currently awaiting final review and merge.
I will update this ticket the moment my PR lands in {{develop}} so that
[~abdelrahman] can pull the latest upstream branch and proceed with the
refactor without any friction.
> Refactor SavingsAccountTransactionsApiResource command dispatch logic
> ---------------------------------------------------------------------
>
> Key: FINERACT-2477
> URL: https://issues.apache.org/jira/browse/FINERACT-2477
> Project: Apache Fineract
> Issue Type: Improvement
> Reporter: saifulhuq
> Priority: Minor
>
> The current implementation of `SavingsAccountTransactionsApiResource` relies
> on a long `else-if` chain for command dispatch.
> As discussed in PR #5465, this should be refactored (possibly using a Command
> Pattern or Switch statement) to improve readability and maintainability.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
