Closed halucinor closed 1 year ago
In GitLab by @henry on Feb 13, 2023, 15:55
The following discussion from !17 should be addressed:
[ ] @henry started a discussion: (+2 comments)
다른 도메인에 대한 컨트롤러에서도, 목록 조회 api를 구현하고 이때 pagination이 들어가게된다면, 위와 비슷한 메소드가 다른 컨트롤러에 똑같이 중복되는 로직이 들어가게될것같네요. 이러한 반복 및 불필요한 로직 구현을 통일 + 다른 개발자가 쉽게 사용할수있도록 개선하는 작업(ex: annotation 구현)이 후에 이루어지면 좋을것같습니다. 지금 당장은 말고, 후에 한번 고민해보면 좋을것같습니다.
다른 도메인에 대한 컨트롤러에서도, 목록 조회 api를 구현하고 이때 pagination이 들어가게된다면, 위와 비슷한 메소드가 다른 컨트롤러에 똑같이 중복되는 로직이 들어가게될것같네요.
pagination
이러한 반복 및 불필요한 로직 구현을 통일 + 다른 개발자가 쉽게 사용할수있도록 개선하는 작업(ex: annotation 구현)이 후에 이루어지면 좋을것같습니다.
annotation
지금 당장은 말고, 후에 한번 고민해보면 좋을것같습니다.
In GitLab by @rkdud1108 on Mar 3, 2023, 17:13
@henry 이전 MR에서 코멘트 남겨주셨던 해당 follow-up issue 관련하여 Custom Annotation을 이용하여 중복 코드를 제거하여 반영하였습니다. !46
In GitLab by @henry on Mar 3, 2023, 18:45
확인했습니다.
고생하셨습니다.
In GitLab by @henry on Feb 13, 2023, 15:55
The following discussion from !17 should be addressed:
[ ] @henry started a discussion: (+2 comments)