-
Notifications
You must be signed in to change notification settings - Fork 5
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
데벨업 v2.0.0 Release #628
Merged
데벨업 v2.0.0 Release #628
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* fix: 사용자가 디스커션에 단 댓글 조회 시, 그 디스커션에 달린 댓글 개수를 잘못 조회하는 오류 수정 * fix: 사용자가 디스커션에 단 댓글 조회 시 테스트 추가 * fix: 사용자가 솔루션에 단 댓글 조회 시, 그 솔루션에 달린 댓글 수 잘못 조회하는 오류 수정 * fix: 사용자가 솔루션에 단 댓글 조회 시, 그 솔루션에 달린 부모 댓글만 댓글 개수에 반영되도록 수정 * fix: 사용자가 디스커션에 단 댓글 조회 시, 그 디스커션에 달린 부모 댓글만 댓글 개수에 반영되도록 수정 * chore: static 키우드 삭제 * chore: fetch join 일반 join으로 변경 * chore: 필요없는 메서드 삭제 * fix: MySolutionComment 에서 필요없는 필드 삭제 * chore: 쿼리 스타일 수정 * chore: 메서드 순서 변경 * chore: 안쓰는 메서드 삭제 * chore: 필요없는 static 키워드 삭제
* refactor: discussion comment parentComment를 parentCommentId로 수정 * refactor: solution comment parentComment를 parentCommentId로 수정 * refactor: findAllByDiscussionIdOrderByCreatedAtAsc n+1 수정 * refactor: findAllBySolutionIdOrderByCreatedAtAsc n+1 수정
Co-authored-by: JEON TAEHEON <brgndyy@gmail.com>
* design: Button defaultText, primaryText 타입 및 디자인 추가 * feat: useSolutionDelete 구현 및 적용 * design: CommentSection margin-top 조절 * feat: 풀이 수정 기능 구현 - 수정하여 진입하는 경우 params 추가 - params 통해 기존 정보 불러오기 - useSolution을 useQuery로 변경, solutionId가 있는 경우만 API 호출하도록 수정 - useSolution 반환 데이터 수정 * feat: 삭제 전 모달 띄우는 기능 추가 * feat: 풀이 글쓴이와 유저 정보가 같을 때만 수정/삭제 버튼 나오도록 구현 * refactor: ModalContent 분리 * rename: SolutionDetailPage 및 컴포넌트 경로 정리 * refactor: 풀이 수정/삭제 컴포넌트 분리 * rename: SolutionBottom을 SolutionDetailBottom으로 변경 * rename: DiscussionDetailPage 및 컴포넌트 경로 정리 * feat: discussion 수정 기능 추가 - useDiscussion을 useQuery로 변경 - patchDiscussion 추가 - DiscussionDetailBottom Component 추가 * refactor: ConfirmButton props의 Confirm 핸들러 이름 변경 * refactor: ConfirmButton props의 Confirm 핸들러 이름 변경에 따른 SolutionDetailBottom props명 변경 * feat: 디스커션 삭제 구현 * fix: SolutionDetailBottom의 userInfo 조건 수정 * design: 디스커션 mission TagButton으로 변경 및 width 100% 지정 * refactor: solutions mock data 및 solution detail msw 핸들러 추가 * refactor: solution delete msw 핸들러 추가 * feat: 솔루션 삭제 시 리스트로 이동하도록 추가 * feat: 디스커션 삭제 시 리스트로 이동하도록 추가
le2sky
assigned le2sky, robinjoon, alstn113, Parkhanyoung, brgndyy, chosim-dvlpr, Minjoo522 and lilychoibb
Oct 9, 2024
alstn113
approved these changes
Oct 9, 2024
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.
파이팅 🫡👍
* refactor: 솔루션 리스트 mock handler 추가 및 mock data의 description url 올바르게 수정 * fix: 풀이 수정 후 제출 시 페이지 이동 기능 추가, msw 풀이 수정 핸들러 추가 * fix: 디스커션 수정 시 데이터 불러와지지 않는 오류 해결 및 discussion msw 연결 * fix: 디스커션 수정 버그 해결 - 디스커션 수정 시 데이터 불러오도록 수정 - 디스커션 수정 시 기존 데이터 불러오도록 수정 - 수정 후 제출 시 수정/새로운 글 분기처리 추가 - Solution이 들어간 네이밍 수정 - 디스커션 불러오기, 삭제, 수정 관련 msw 핸들러 추가 * design: 디스커션 작성하기 버튼이 항상 오른쪽에 오도록 수정 --------- Co-authored-by: Haneul Lee <leehaneul990623@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
구현 요약
화이팅