Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

도메인 분석 후 도출된 이슈들에 대해서 회의한다. #1340

Open
nuyh99 opened this issue Jun 5, 2023 · 0 comments
Open

도메인 분석 후 도출된 이슈들에 대해서 회의한다. #1340

nuyh99 opened this issue Jun 5, 2023 · 0 comments
Labels

Comments

@nuyh99
Copy link
Contributor

nuyh99 commented Jun 5, 2023

Prolog 도메인을 전체적으로 분석한 이후 아래의 이슈들을 정리했습니다.
기존 생성된 이슈와 중복된 것들은 제거하고, 어떤 식으로 개선할 지 회의가 필요합니다.

  • admin only 주석 어떻게 처리할 지??
  • updateMember()에서 이미지 변경이 안됨
  • 쿠키를 사용해서 중복 조회수를 관리하고 있었는데 현재 사용되고 있지 않음
  • 조회수 관련 주작 막을 지 말지
  • 로그인 유저가 자신이 읽었던 글을 DB의 테이블에 저장하는데, 쿠키 등의 다른 방법이 낫지 않을까?
  • StudylogService.deleteStudylog()에서 마지막 이벤트 발행 -> 이거 쓰이는 건지?
  • StudylogService.findStudylogs() 의 오버로딩 버전 중 하나가 읽기&쓰기 트랜잭션이 달려있는데, 내부 호출을 하고 있기 때문에 트랜잭션이 적용되지 않음
  • 검색 시 동적 쿼리 부분을 코드로 관리하는데 이것을 ES로 넘기기
  • Tag 생성할 때 한글 또는 영어만 쓰고 ‘#’은 서버 측에서 붙여서 저장하는 것이 어떤지?
  • MemberGroup, GroupMember 관련 -> 새 회원을 추가할 때 그룹을 지정하도록 하는 게 나을듯
  • ES 5기부터 안됨
  • findAllWithMySessionFirst(), findAllWithMyMissionFirst()가 있는데 안됨
  • Role의 사용처?? 안 쓰이는 걸로 보임 (JWT Token 에서도 Claim으로 있지만 사용되나??)
  • 로드맵의 퀴즈 관련 기능은 admin only이지만 적용돼있지 않음
  • SessionService, NewSessionService 둘의 존재 이유?
  • KeywordService도 admin only 아닌가요??
  • validateToken에서 NumberFormatException을 캐치하는데 터질 이유가 있나요?? 토큰의 Subject는 Member 테이블의 PK 값만 들어오는 상태인데…
  • CustomException만 추가해두고 BadRequest 측의 Enum에는 메세지 추가 안해뒀음
@nuyh99 nuyh99 added the BE label Jun 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant