AshharAhmadKhan commented on PR #5498: URL: https://github.com/apache/fineract/pull/5498#issuecomment-3939308945
Hi @IOhacker, Thank you for the feedback — I've consolidated everything into a single commit as requested. This PR now removes the **entire insecure self-service feature** in one commit: - All self-service API controllers and Swagger docs (~18 controllers) - All self-service service layer implementations - All self-service domain entities, repositories, and exceptions - `AppUserClientMapping` entity and all references in `AppUser.java` - `SelfServiceUserAuthorizationManager` from the security module - `DeviceRegistration` feature (GCM/push notification device tracking) - Pocket feature (self-service account grouping) - Self-service wiring cleaned up in `FineractFeignClient`, `NotificationSenderService`, `SmsCampaignValidator`, `UserAdministrationConfiguration` - Liquibase migration `0210` to drop associated database tables **Stats:** 135 files changed, 10,836 deletions All modules compile clean and `spotlessApply` passes. Please let me know if anything else needs adjusting. -- 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]
