[Feat] 어드민 API 연동 #159

jhj2713 · 2024-08-16T01:24:06Z

🖥️ Preview

close #148

✏️ 한 일

선착순 API 연동
- 선착순 이벤트 목록 조회
- 선착순 이벤트 목록 수정
- 참여자 목록 조회
- 당첨자 목록 조회
- 밸런스 게임 선택지 옵션 조회
추첨 API 연동
- 추첨 이벤트 조회
- 추첨 이벤트 수정
- 참여자 목록 조회
- 당첨자 목록 조회
- 기대평 조회
- 기대평 삭제

❗️ 발생한 이슈 (해결 방안)

Modal 내부 무한 스크롤이 발생하지 않는 문제가 있어서 요렇게 해결했습니당 >> wiki

❓ 논의가 필요한 사항

github-actions · 2024-08-16T01:24:47Z

빌드를 성공했습니다! 🎉

github-actions · 2024-08-16T04:25:26Z

빌드를 성공했습니다! 🎉

sooyeoniya

이미지 업로드랑 무한스크롤 작업 하느라 고생많으셨습니다!! 🥹
타입스크립트 정말 잘하시는 것 같아용,, 많이 배워갑니다 :)

sooyeoniya · 2024-08-18T09:36:00Z

admin/src/features/Rush/EventList.tsx

+    const handleChangeDate = (changeIdx: number, newDate: string) => {
+        const selectedItem = rushList[changeIdx];
+        const selectedTime = selectedItem.startTime || "00:00";
+        const selectedDateTime = new Date(`${newDate}T${selectedTime}`).getTime();
+        const currentTime = new Date().getTime();
+        const endDateTime = new Date(`${newDate}T${selectedItem.endTime || "23:59"}`).getTime();


new Date().getTime() -> 몇군데 보이는데 여기두 client에서 만든 getMsTime 유틸 적용 가능할까여?

de33a99 오호 좋아용 반영했습니다!

sooyeoniya · 2024-08-18T15:03:17Z

admin/src/features/Rush/RushPrizeForm.tsx

+        if (selectedFile instanceof File) {
+            const formData = new FormData();
+            formData.append("image", selectedFile);
+            postImage(formData);


요기 비동기함수 같은데 await 붙이지 않은 이유가 있는걸까용?
RushSelectForm에서는 postImage 호출할 때 await 붙어있는 것 같아서여!

동기적으로 동작할 필요가 없는 곳에서는 await 붙일 필요가 없으니까 안 붙여줬어요! RushSelectForm에서 postImage 할 때도 안 붙여도 되는데 코드 짜다가 잘못 들어갔나보네욥,,ㅎ

sooyeoniya · 2024-08-18T15:35:24Z

admin/src/features/Rush/RushSelectForm.tsx

+            .map((file, index) => {
+                if (!(file instanceof File)) {
+                    return null;
+                }
+                const formData = new FormData();
+                formData.append("image", file);
+                return { formData, index };
+            })
+            .filter((item): item is { formData: FormData; index: number } => item !== null);


저희 전에 index를 idx로 맞추기로 했었는데, 혹싀 index라고 쓴 이유가 있나용??

어드민이라고 별 생각없이 했나봐욥,, 수정했습니다! f0b236d

sooyeoniya · 2024-08-18T16:14:32Z

admin/src/hooks/useFetch.ts


-export default function useFetch<T, P = void>(fetch: (params: P) => Promise<T>, showError = true) {
+export default function useFetch<T, P = void>(
+    fetch: (params: P, token: string) => Promise<T>,


client에서는 토큰 인자를 따로 넣지 않았던 것 같은데 여기는 넣어준 이유가 뭔가용?? admin에서는 항상 인증 토큰이 필요하기 때문인가요??

넵 어드민은 로그인 API 제외하고는 인증 토큰이 필요해서 넣어줬어용

sooyeoniya · 2024-08-18T16:38:26Z

admin/src/utils/validateDateTime.ts

+            return "시작 날짜와 시간은 현재 시간보다 빠를 수 없습니다!";
+        }
+        if (startDateTime > endDateTime) {
+            return "시작 날짜와 시간이 종료 날짜와 시간보다 빠를 수 없습니다!";


"시작 날짜와 시간이 종료 날짜와 시간보다 느릴 수 없습니다!" 이지 않을까용?

sooyeoniya · 2024-08-18T16:43:41Z

admin/src/utils/validateDateTime.ts

+
+    if (key === "endDate" || key === "endTime") {
+        if (endDateTime < startDateTime) {
+            return "종료 날짜와 시간이 시작 날짜와 시간보다 느릴 수 없습니다!";


if (startDateTime < currentDateTime) { return "시작 날짜와 시간은 현재 시간보다 빠를 수 없습니다!"; }

위 문구가 맞는 말이라면 이 부분도 아래처럼 고쳐야지 않을까 싶네용
(endDateTime < startDateTime) -> 여기서 종료 부분이 시작 부분보다 숫자가 빠르다고 설정되어있으니까 아래처럼 쓰는게 맞을 것 같아영
"종료 날짜와 시간이 시작 날짜와 시간보다 빠를 수 없습니다!"

말해준 부분들 둘 다 수정했어요! 31f8bf3

sooyeoniya · 2024-08-18T17:47:59Z

admin/src/hooks/useInfiniteFetch.ts

@@ -45,10 +57,9 @@ export default function useInfiniteFetch<T>({
            const lastPage = await fetch(currentPageParam);
            const nextPageParam = getNextPageParam(currentPageParam, lastPage);

-            setData([...data, ...lastPage.participants]);
+            setData([...data, ...getDataList(lastPage)]);


setData(prevData => [...prevData, ...getDataList(lastPage)]);
예전에 효정님 코드가 생각났는데 좀 더 정확한 상태 유지를 위해서 prevData 써서 이전 상태 기반으로 업데이트 해주면 어떨까용?

이거 요런식으로 고치면 처음에 불러오는 page 0 데이터가 두 번 세팅이 되더라구용,,? 처음에 렌더링이 두 번 되면서 생기는 현상인 것 같아서 일단 함수형 업데이트는 안 써줬어용

헉 그렇군용,, 좋습니다 !! :)

sooyeoniya · 2024-08-18T18:01:05Z

admin/src/hooks/useInfiniteFetch.ts

+    const getDataList = (lastPage: R) => {
+        if (isInfiniteParticipantListData<T>(lastPage)) {
+            setTotalLength(lastPage.totalParticipants);
+            return lastPage.participantsList;
+        } else if (isInfiniteExpectationListData<T>(lastPage)) {
+            setTotalLength(lastPage.totalExpectations);
+            return lastPage.expectations;
+        }
+
+        return [];
+    };


여기 getDataList도 useCallback로 묶어주는건 어떤가용?

좋은 생각이에욤 반영했습니당 69886df

sooyeoniya · 2024-08-18T18:15:08Z

admin/src/pages/LotteryParticipantList/index.tsx

LotteryParticipantList랑 LotteryWinnerList에 중복 코드가 매우 많은데 공통으로 분리해서 사용할 수 있는 방법도 고려해보면 좋을 것 같아여!

github-actions · 2024-08-19T01:40:51Z

빌드를 성공했습니다! 🎉

jhj2713 added 27 commits August 13, 2024 15:45

feat: 선착순 밸런스 게임 이벤트 불러오기 API 연동

14ee6fc

fix: API에 맞게 option 수정

86e1e63

feat: 추첨 이벤트 조회 API 연동

32600fa

fix: 추첨 이벤트 수정 API 요청 수정

31d1dac

fix: participants -> participantsList

b3592bd

feat: 이미지 업로드 로직 구현

cf6d678

fix: 타입 수정 및 불필요한 구문 삭제

77d589a

feat: 이미지 API 연동

8d9e6d3

fix: 이미지 업로드 조건 수정

5c8a356

feat: 무한 스크롤 params 수정

416be8e

feat: 이벤트 진행 날짜 / 시간 예외처리 추가

c868c4f

feat: 추첨 이벤트 수정 API 반영

91ff881

feat: 수정 API 연동

a751bcb

feat: 기대평 삭제 로직 구현

7e2ad0c

fix: infinite 조회 수정

d4684a3

fix: token 로직 수정

067550f

feat: 기대평 무한 스크롤

11bc298

fix: API 요청 칼럼 맞추기

6972546

fix: start fetching 조건 추가

82f0158

feat: 오류 시 cookie 제거 로직 구현

787597d

fix: intersection observer 수정

1466aa5

fix: memo 적용

3b40b7d

fix: 불필요한 props 삭제

85ff362

refactor: useMemo 적용

859f1f0

fix: ModalComponent가 isOpen 상태에 따라 계속 리렌더링되는 현상 방지

f02c18a

chore: header에 root 라우팅 적용

265fd51

feat: 추첨 API 연동

8d02920

jhj2713 added the feat 기능 구현 label Aug 16, 2024

jhj2713 requested a review from sooyeoniya August 16, 2024 01:24

jhj2713 self-assigned this Aug 16, 2024

fix: participant fetch 조건 수정

8e0afed

sooyeoniya approved these changes Aug 18, 2024

View reviewed changes

jhj2713 added 5 commits August 19, 2024 10:07

refactor: getMsTime util 적용

de33a99

chore: 필요없는 await 삭제

bb37f19

chore: index -> idx

f0b236d

chore: validate 문구 수정

31f8bf3

chore: useCallback 추가

69886df

sooyeoniya merged commit 59b01f0 into dev Aug 19, 2024
2 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Feat] 어드민 API 연동 #159

[Feat] 어드민 API 연동 #159

jhj2713 commented Aug 16, 2024 •

edited

Loading

github-actions bot commented Aug 16, 2024

github-actions bot commented Aug 16, 2024

sooyeoniya left a comment

sooyeoniya Aug 18, 2024

jhj2713 Aug 19, 2024

sooyeoniya Aug 18, 2024

jhj2713 Aug 19, 2024

sooyeoniya Aug 18, 2024

jhj2713 Aug 19, 2024

sooyeoniya Aug 18, 2024

jhj2713 Aug 19, 2024

sooyeoniya Aug 18, 2024

sooyeoniya Aug 18, 2024

jhj2713 Aug 19, 2024

sooyeoniya Aug 18, 2024

jhj2713 Aug 19, 2024

sooyeoniya Aug 19, 2024

sooyeoniya Aug 18, 2024

jhj2713 Aug 19, 2024

sooyeoniya Aug 18, 2024

github-actions bot commented Aug 19, 2024

[Feat] 어드민 API 연동 #159

[Feat] 어드민 API 연동 #159

Conversation

jhj2713 commented Aug 16, 2024 • edited Loading

🖥️ Preview

✏️ 한 일

❗️ 발생한 이슈 (해결 방안)

❓ 논의가 필요한 사항

github-actions bot commented Aug 16, 2024

github-actions bot commented Aug 16, 2024

sooyeoniya left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

github-actions bot commented Aug 19, 2024

jhj2713 commented Aug 16, 2024 •

edited

Loading