랭킹페이지 페이지네이션 API 연결 구현 완료 by xhmxnjxx · Pull Request #26 · team-tadak/tadak-frontend

xhmxnjxx · 2022-11-18T04:40:35Z

Related to #11

체크 리스트

적절한 제목으로 수정했나요?
상단에 이슈 번호를 기입했나요?
Target Branch를 올바르게 설정했나요?
Label을 알맞게 설정했나요?

작업 내역

랭킹 페이지 페이지네이션 API 연결 구현

비고

버그: 페이지네이션에서 처음 페이지를 누를 때는 랭킹보드가 제대로 뜨는데 다시 그 페이지를 누르면 1위와 2위를 제외한 최상위 한 명의 리더보드만 나옴

vercel · 2022-11-18T04:40:36Z

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated
idea-test	✅ Ready (Inspect)	Visit Preview	Nov 19, 2022 at 6:52PM (UTC)

jaychang99 · 2022-11-18T06:14:23Z


  // c
-  "#include <stdio.h>\n\n int main(){\n\n    printf(“Hello world” );\n    return 0;\n}",
+  "#include <stdio.h>\n\nint main() {\n  printf(“Hello world”);\n  return 0;\n}",


앗 미세한 오타들을 잘 잡아주셨네요! 이 라인에 printf("Hello world!") 를 보시면 큰따옴표가 양쪽 모양이 다른 일반적인 문서의 큰따옴표로 돼 있어서 실제 입력할 때는 큰따옴표를 입력해도 오타로 인식하더라구요. 아마 요 부분 그냥 코드에디터 상에서 지웠다가 큰 따옴표 입력해주시면 고쳐질 것 같아요!

네네 수정해보겠습니다!

jaychang99 · 2022-11-18T06:16:40Z

+  const [offSet, setOffSet] = useState(2);
+
+
+  const topRanks = useRanks(1, pageSize, languageNo, grammarNo).ranks;


앗 topRanks 같은 경우 고정적으로 pageSize 2로 해서 받아와도 될 것 같은데 어떻게 생각하시나용

앗 그러네요! 반영해서 수정해보도록 하겠습니다!

Feature #11 modal issues

…/github.com/jaychang99/tadak-frontend into Feature_Issue#11_Leaderboard_Pagination_API

jaychang99 · 2022-11-19T18:06:26Z

          ranks.map((rank, index) => (
            <LeaderBoardItem
-              rank={(pageNumber == 1) ? index + 1 + TOP_DISPLAY_USER : index + 1 + pageSize}
+              rank={rank.ranking}


오홍 맞네요 이제 API 자체에서 ranking 을 주니까 pageSize 와 pageNumber 로 계산을 안 해도 돼서 좋군요. 그러면 prop 에서도 받아오지 않아도 좋을 것 같은데 어떻게 생각하시나용?

앗 그러네요 ,,, 수정할게요 !

확인했습니다 본 thread 는 resolve 하겟습니다!

jaychang99 · 2022-11-19T18:14:21Z

 import React, { useState, useEffect } from "react";
 import { defaultFadeInVariants, staggerHalf, staggerQuarter } from "styles/motions";
-import { filterProps, motion } from "framer-motion";
+import { motion } from "framer-motion";


지금 보니까 이 motion 도 아래에서 안 쓰이는 것 같은데 확인 한번 해 주실 수 있나용 이건 아마 제가 import 해둔거같긴한데... ㅜ.ㅜ 헛헛

저도 처음에 그런 줄 알고 지웠다가 오류가 나서 봤더니 motion.div에서 쓰이는 것 같더라구요 ,,,,

앗 맞네요 밑에 <motion.div> 가 있네요..헛헛 😅 코드 가 접혀져서 못 봤나보네요 죄송함니다...ㅋㅋ

jaychang99 · 2022-11-19T18:19:02Z

+            {ranks.length !== 0 && (
              <> 
-                <LeaderBoard ranks={(pageNumber == 1) ? ranks.slice(2) : ranks} pageNumber={pageNumber} pageSize={pageSize}></LeaderBoard>
+                <LeaderBoard ranks={(pageNumber === 1) ? ranks.slice(2) : ranks} pageNumber={pageNumber} pageSize={LEADERBOARD_PAGE_SIZE}></LeaderBoard>


API 에 이제 ranking 이 추가되어서 요기 이제 pageNumber 랑 pageSize 넘기지 않아도 되지 않을까용?

네네 안 남겨도 될 것 같습니다

jaychang99 · 2022-11-19T18:25:22Z

늦은 시간에도 고생하셨습니다! 추가적인 부분도 comment 달아놓았습니다!

xhmxnjxx · 2022-11-19T18:54:05Z

늦은 시간에도 고생하셨습니다! 추가적인 부분도 comment 달아놓았습니다!

src/pages/Login/index.js 에서 try-catch 부분 제외하고 수정 완료해서 push 해두었습니다!

jaychang99

LGTM 😄 수고많으셨씁니다!

jaychang99 · 2022-11-20T02:07:57Z

 import React, { useState, useEffect } from "react";
 import { defaultFadeInVariants, staggerHalf, staggerQuarter } from "styles/motions";
-import { filterProps, motion } from "framer-motion";
+import { motion } from "framer-motion";


앗 맞네요 밑에 <motion.div> 가 있네요..헛헛 😅 코드 가 접혀져서 못 봤나보네요 죄송함니다...ㅋㅋ

jaychang99 · 2022-11-20T02:08:52Z

          ranks.map((rank, index) => (
            <LeaderBoardItem
-              rank={(pageNumber == 1) ? index + 1 + TOP_DISPLAY_USER : index + 1 + pageSize}
+              rank={rank.ranking}


확인했습니다 본 thread 는 resolve 하겟습니다!

xhmxnjxx added 6 commits November 18, 2022 03:24

python 상수 오탈자 수정

72dffc5

html 상수 오탈자 수정

3e18e6d

c 상수 오탈자 수정

a5da9b8

로그인 된 상태에서 로그인 화면 접속 시도 시 메인 화면으로 redirect 하는 기능 구현

8fc1331

Leaderboard Pagination 전체 페이지 구현

9313565

리더보드 페이지 페이지네이션 API 연결 구현

4d5db8d

xhmxnjxx added the feature 기능 추가 label Nov 18, 2022

xhmxnjxx closed this Nov 18, 2022

xhmxnjxx reopened this Nov 18, 2022

hajeong-lim and others added 2 commits November 18, 2022 14:41

fix: Change || to ?? operator

4796142

랭킹보드에서 2페이지 넘어갈 시 Leaderboard 순위 버그 수정

b87ca20

vercel Bot deployed to Preview November 18, 2022 05:48 View deployment

hajeong-lim added 4 commits November 18, 2022 14:49

fix: Remove unnecessary theme imports

2968d6e

fix: Rename PortalModal/style.js to styles.js

1d56863

fix: Move common styles in PortalModal to component/common/styles.js

05a55c8

fix: Fix Modal style imports

e1361ed

jaychang99 suggested changes Nov 18, 2022

View reviewed changes

hajeong-lim and others added 3 commits November 19, 2022 13:11

fix: Renmae style.js to styles.js

581e50d

fix: Chagne import path in PortalModal

a1624df

Merge pull request #24 from jaychang99/Feature_#11_Modal_issues

8c0c397

Feature #11 modal issues

jaychang99 mentioned this pull request Nov 19, 2022

로그인 및 회원가입 페이지 레이아웃 수정, C언어 상수 오탈자 처리 #23

Closed

4 tasks

xhmxnjxx added 2 commits November 20, 2022 01:09

LEADERBOARD_PAGE_SIZE 상수 처리

4faedc4

API 구조 변경에 따른 코드 변경 및 develop branch merge conflict 처리

72c4ef3

vercel Bot deployed to Preview November 19, 2022 17:26 View deployment

xhmxnjxx requested a review from jaychang99 November 19, 2022 17:28

const {offset, setOffset} 삭제

88675b0

vercel Bot deployed to Preview November 19, 2022 17:34 View deployment

주석 처리

8932ecb

vercel Bot deployed to Preview November 19, 2022 17:43 View deployment

xhmxnjxx added 2 commits November 20, 2022 03:19

.env 파일 추가

d774186

Merge branch 'Feature_Issue#11_Leaderboard_Pagination_API' of https:/…

6186ea3

…/github.com/jaychang99/tadak-frontend into Feature_Issue#11_Leaderboard_Pagination_API

jaychang99 suggested changes Nov 19, 2022

View reviewed changes

Leaderboard props에서 pageNumber, pageSize 삭제

82c0e44

vercel Bot deployed to Preview November 19, 2022 18:51 View deployment

console.log 부분 삭제

8604015

vercel Bot deployed to Preview November 19, 2022 18:52 View deployment

xhmxnjxx requested a review from jaychang99 November 19, 2022 18:54

jaychang99 approved these changes Nov 20, 2022

View reviewed changes

jaychang99 requested a review from skkuchoi November 20, 2022 02:10

jaychang99 mentioned this pull request Nov 22, 2022

(배포 전) 필수 기능 고치기 및 추가 구현 #11

Open

52 tasks

jaychang99 force-pushed the develop branch from 8c0c397 to 00f1658 Compare June 5, 2023 11:18

		const [offSet, setOffSet] = useState(2);


		const topRanks = useRanks(1, pageSize, languageNo, grammarNo).ranks;

Uh oh!

Conversation

xhmxnjxx commented Nov 18, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

체크 리스트

작업 내역

비고

Uh oh!

vercel Bot commented Nov 18, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

jaychang99 commented Nov 19, 2022

Uh oh!

xhmxnjxx commented Nov 19, 2022

Uh oh!

jaychang99 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

xhmxnjxx commented Nov 18, 2022 •

edited

Loading

vercel Bot commented Nov 18, 2022 •

edited

Loading