Feat: Detail페이지 스터디상세api 연결 및 postApplyStatus 에 따른 버튼 맵핑 구현 #174
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
모든 모달 컴포넌트의 닫히는 함수를 onClose로 일관되게 통일. onClose와 동일한 기능을 하는 onClick prop을 제거
kimbobby
requested review from
nakyoung98,
JIS0098,
hankim0904 and
Yoon-kyungLee
올바른 PR 형식올바른 PR 형식입니다. |
kimbobby
commented
Mar 26, 2024
hankim0904
approved these changes
Mar 27, 2024
JIS0098
approved these changes
Mar 27, 2024
nakyoung98
approved these changes
Mar 27, 2024
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.
📌요구사항
📌작업 진행 상황 (에러, 막혔던 부분, 그외 궁금한것 등등)
댓글 목록 불러오기, 작성, 수정, 삭제, 스터디 삭제 연결해야합니다••• •• 🙃
기타 사항
유저 프로필 클릭시 유저정보모달이 뜨려면 id가 필요하지만 현재 서버에서 id를 내려주고 있지 않아 UserInfo에 id 를 잠시 undefined도 받을 수 있도록 처리해놨습니다..
모달에 onClick이랑 onClose랑 같은 기능을 하는 함수로 판단되어 onClose로 통일 해서 수정했습니다.
모달레이아웃이 수정되기전 유저응답모달을 구현했어서 css수정이 조금 필요해 margin 조금 수정했습니다.
스터디 상세카드 기술스택 부분 align center 맞춰야할거같습니다. 그리고 기타로 표시되어도 링크 아이콘이 뜨네용 나경님이 링크 관련해서 수정하시고 계시니 따로 만지진 않았습니다..!
📌스크린샷 / 테스트결과 (선택)
스터디상세api 연결
postApplyStatus === OWNER
postApplyStatus === NOT_APPLIED
2024-03-27.4.45.44.mov
postApplyStatus === APPLIED
2024-03-27.4.44.23.mov
postApplyStatus === INVITED
2024-03-27.5.59.01.mov
postApplyStatus === RECRUIT_CLOSED
📌이슈 번호
#173