AshharAhmadKhan opened a new pull request, #5879: URL: https://github.com/apache/fineract/pull/5879
JIRA: https://issues.apache.org/jira/browse/FINERACT-2571 The original goal of this ticket was to migrate NotificationHelper to use the fineract-client (Feign) instead of the legacy Utils.performServerGet REST calls. After investigating the current state of develop, the migration has already been completed via a different approach. FeignNotificationHelper was introduced as the proper typed replacement using FineractFeignClient, and NotificationApiTest was fully rewritten to use it directly. The old NotificationHelper class was marked @Deprecated(forRemoval = true) but never actually removed. Checked the codebase and nothing references NotificationHelper anymore, so this PR removes it as the clean resolution of this ticket. Prior draft PRs (#5703 and #5713) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
