-
Notifications
You must be signed in to change notification settings - Fork 0
문제 등록 방식 변경, 해설 썸네일 정보 제공 #105
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
Conversation
- 해설 및 홀드 id 검증 로직은 추후 구현
ChoiWonYu
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
현재 빠르게 구현하기 위해선 이 구조가 최선인 것 같긴 하네요..!
현재 애매한 부분을 기록해놨다가 추후에 만났을 때 개선된 구조를 논의해보는 게 좋을 것 같습니다.
첫 번째로 문제로 등록되는 solution 생성 API 비즈니스 로직을 현재 API 서버로 이전하는 것이 가장 급할 것 같네요.
두 번째로 thumbnailSolutionId와 holdId가 Path Param이 아닌 Query Param으로 받고 있는데 Hold의 계층 구조가 애매하지만, Path Param으로 처리하는 것이 더 자연스러워 보이긴 합니다. 이 부분도 논의를 해보면 좋을 것 같아요..!
업로드 서버 구현이 끝나시고 시간 괜찮으시면 클라이언트, 업로드 서버, API 서버 간 문제를 생성하는 해설 업로드, 생성된 문제에 추가되는 해설 업로드 과정에서의 API 흐름 도식화해서 기술 부채로 티켓 추가해주시면 감사하겠습니다..!🙇♂️ 고생하셨습니다 👍
src/main/java/com/first/flash/climbing/solution/application/SolutionService.java
Outdated
Show resolved
Hide resolved
ChoiWonYu
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
썸네일 이미지 변경이 문제에 반영되는 것 확인했습니다👍
요약
첫 해설 등록 시 문제를 함께 생성하기 위해 로직 변경
내용
클라이밍장 상세 정보 조회 시 sectorId 정보 제공
GET /gyms/{gymId}Solution에 풀이 일자, 썸네일 url 추가
problem에 홀드 정보, 썸네일 해설 정보 추가
문제 생성을 일반 유저도 할 수 있도록 변경
POST /admin/gyms/{gymId}/sectors/{sectorId}/problemsPOST /gyms/{gymId}/sectors/{sectorId}/problems새로운 해설 업로드 시 작업 순서
기존에 있는 문제 업로드 API 변경
없는 유저의 해설 업로드
기존 문제에 해설 업로드
새로운 문제를 생성하며 해설 업로드
업로드된 해설 영상 수정
해설 삭제 로직 변경
마이페이지의 해설 조회 시 제공 정보 변경
같은 섹터, 난이도, 홀드색을 가진 문제가 있는지 조회하는 API 구현