feat: 이전 모집공고 리스트 출력 #372
feat: 이전 모집공고 리스트 출력 #372
Conversation
|
수고많으셨습니다. |
| ? (recruitHistories.data?.recruitments ?? []) | ||
| : []; | ||
|
|
||
| if (!(await searchParams).rid) { |
There was a problem hiding this comment.
이 부분을 반복적으로 await를 호출해서 처리하고 있는 것으로 보이는데요, 초기에 tab를 받아올때 구조분해 할당을 통해서 한번에 가져오는게 더 깔끔할 거 같다고 생각됩니다!
const { tab = 'recruit', rid} = await searchParams;이렇게요!
추가로 rid는 recruitmentId를 뜻하는 거 같은데 쿼리스트링을 사용할 생각이셔서 짧게 한거 같긴한데 그외에는 좀 더 명확한 네이밍을 가지면 좋을거 같아요.
|
아 그리고 이전 다음 을 눌렀을때 넘어가는 동작이 부자연스럽게 느껴지는 것 같아요 |
sins051301
left a comment
sins051301
left a comment
There was a problem hiding this comment.
수고하셨습니다 지우님도 변수명이나 함수명 쓰실 때 더 직관적으로 설계해주시면 좋을 것 같아요
| selectedRid: number; | ||
| } | ||
|
|
||
| function usePageSize() { |
There was a problem hiding this comment.
여기에서만 쓰이는 훅이면 함수명에 특정 도메인 붙여주면 좋을 것 같아요
다른 곳에서도 쓰일 수 있다면 파일로 분리해주세용
| default: | ||
| return `/club/${id}`; | ||
| } | ||
| const qs = new URLSearchParams(); |
|
|
||
| const [page, setPage] = useState(0); | ||
| const start = page * pageSize; | ||
| const visible = list.slice(start, start + pageSize); |
There was a problem hiding this comment.
visible이 구체적으로 어떤것을 보여주는지 변수명 수정해 주면 좋을 것 같아요
| redirect(`/club/${id}?${qs.toString()}`); | ||
| } | ||
|
|
||
| const rid = Number((await searchParams).rid) || recent.data.id; |
There was a problem hiding this comment.
rid가 구체적으로 무엇을 나타내는지 모르겠어요
#️⃣연관된 이슈
📝작업 내용
스크린샷 (선택)
2026-01-05.12.56.37.mov
💬리뷰 요구사항(선택)
지금 헤더 부분에 대해서는 기존대로 뒀는데 저희가 헤더에 모집공고, 모집글 작성 날짜가 나오고 있어서 이 부분을 그냥 둘지 아니면 모집공고가 변함에 따라 해당 모집공고에 대한 날짜로 변경을 할지 의견 여쭤봅니다 !
제가 고민인 부분은 모집공고에 맞춰서 상단 헤더가 같이 변경이 되면 자연스럽긴한데 그러면 모집중, 모집마감도 함께 변경이 되니... 이 부분이 괜찮을지 고민입니다.