전남대_13조_fundering_9주차 #121
Merged
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
chore: Dockerfile 생성
chore: Dockerfile yarn으로 변경
펀딩 후원하기 페이지 UI, 포트원 연동, 펀딩 정보 조회 api
셀럽 팔로우, 언팔로우 / 셀럽신청 mock api 작업
로그인 여부 및 관리자 여부에 따른 접근제한
fix: 충돌 이후 router 수정
9주차 develop -> master
|
안녕하세요 13조 여러분..! 이번 주도 고생 많으셨습니다 :) 마지막 까지 화이팅 입니다..! |
dowookims
reviewed
Nov 6, 2023
| method: "GET", | ||
| params: { pageIndex: pageIndex, keyword: keyword, sortType: sortType }, | ||
| }); | ||
| }; |
There was a problem hiding this comment.
지금 인자로 받아서 그대로 다른 객체에 넘겨주게 될 때 동일한 인터페이스를 사용한다면
props 이름을 params 처럼 건네주는 object의 키 값이랑 동일하게 해서 객체를 간결하게 사용�하는건 어떨까요?
dowookims
reviewed
Nov 6, 2023
|
|
||
| useEffect(() => { | ||
| componentNumberRef.current = calculateComponentCount(windowWidth); | ||
| }, [windowWidth]); |
There was a problem hiding this comment.
여기서 ref를 사용하는 이유가 있을까요? 굳이 ref를 사용하지 않고 resize가 일어날 때 setState를 calculated된 값으로 해도 될 것 같아요.
|
다들 이제 잘작업 해 주고 있으셔서, 그리고 대게 UI 작업이어서 그런지 잘해주고 있으신거 같습니다. 코드 상으로는 크게 특이점은 없는것 같은데, 작업 진행하면서 막히거나 조금더 피드백 받고 싶은 부분이 있으시면 멘토링때 나누면 좋을 것 같아요. 고생 많으셨습니다. |
dowookims
approved these changes
Nov 6, 2023
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.
안녕하세요 멘토님! 이번주는 mock api를 활용해 api 핸들러 함수와 api 관련 훅, 스켈레톤과 로더, Suspense 관련 작업을 주로 진행하였습니다. 시간이 부족해진 관계로 급하게 코드를 작성하고 있어서 코드가 점점 더러워지는 것 같네요. ㅠㅠ
아직 백엔드 배포가 되지 않았고 화요일 전까지 주요 기능 배포할 예정이라고 해서 기다리며 mock api를 활용 중입니다.
react-query의 onError, onSuccess의 경우 일단은 지금처럼 진행하고 리팩토링을 계획 중입니다.
현재 대부분의 UI 작업이 끝났지만 아직 아래의 UI를 작업 중입니다. 순서는 우선순위와 같습니다. 아직 꽤 남았지만(ㅠㅠ) 공통 컴포넌트를 활용하는 페이지가 꽤 있어서 마감 전에 끝나는 것을 목표로 열심히 작업 중입니다.
멘토링 일지는 링크에서 확인 가능합니다.
진행 완료한 PR
#91
#95
#96
#101
#104
#105
#99
#107
#98
#109
#112
#113
#110
#116
진행 중인 PR 및 이슈
#115
#114
#108
#118
#119