You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
There are places where a proper code paradigm is not followed, such as dependency injection, and certain places where the components are strongly coupled, making testing them difficult. This ticket tracks all these code changes that are required, individual tickets should be filed for each of these issues. This issue will be a long running thread and will be updated as and when we find code that needs to be cleaned up.\n\n- [x] NewNetcoreService needs to be fetched using dependency injection instead of using getInstance and actually creating an instance using the new keyword.\n- [ ] All the services using NetcoreWhatsapp currently manually set some of the fields like FileCdn etc. We need to autoconfigure those.\n- [ ] Currently, the RedisCache change to remove @AllArgsConstructor was rolled back due to null pointer exceptions. We need to pinpoint the point of failure and fix this.\n\n\n### Project\nUCI\n\n\n### Organization Name:\nSamagra\n\n### Domain\nBackend\n\n### Area of governance \nCommunications\n\n\n### Tech Skills Needed:\nJava, SpringBoot\n\n### Mentor(s)\n@chinmoy12c \n\n### Complexity\nMedium\n\n### Category\nCode Cleanup\n\n### Sub Category\nAPI, Backend, Technical Debt.
The text was updated successfully, but these errors were encountered:
There are places where a proper code paradigm is not followed, such as dependency injection, and certain places where the components are strongly coupled, making testing them difficult. This ticket tracks all these code changes that are required, individual tickets should be filed for each of these issues. This issue will be a long running thread and will be updated as and when we find code that needs to be cleaned up.\n\n- [x] NewNetcoreService needs to be fetched using dependency injection instead of using getInstance and actually creating an instance using the
new
keyword.\n- [ ] All the services using NetcoreWhatsapp currently manually set some of the fields like FileCdn etc. We need to autoconfigure those.\n- [ ] Currently, the RedisCache change to remove @AllArgsConstructor was rolled back due to null pointer exceptions. We need to pinpoint the point of failure and fix this.\n\n\n### Project\nUCI\n\n\n### Organization Name:\nSamagra\n\n### Domain\nBackend\n\n### Area of governance \nCommunications\n\n\n### Tech Skills Needed:\nJava, SpringBoot\n\n### Mentor(s)\n@chinmoy12c \n\n### Complexity\nMedium\n\n### Category\nCode Cleanup\n\n### Sub Category\nAPI, Backend, Technical Debt.The text was updated successfully, but these errors were encountered: