Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

➕ [WV-35] feat : player - 투수 리스트 페이지 생성 #43

Merged
merged 9 commits into from
Jan 9, 2025

Conversation

hayoung78
Copy link
Contributor

개요

player - 투수 리스트 페이지 생성


PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • PR 제목 및 커밋 메시지 컨벤션 확인
  • 직접 만든 함수가 있다면 이에 대한 설명 추가 (ex. JS DOCS)
  • 변경 사항에 대한 테스트 완료 (버그 수정/기능에 대한 테스트)
  • Label 확인
  • Assignees 설정 확인
  • Reviewers 설정 확인

PR details

투수 리스트 페이지 생성

  • 코치리스트 페이지때 만든 코치카드 -> playerCard 로 변경 후 재사용
  • api 함수 생성
export const getPitcherList = async (): Promise<PitcherInfo[]> => {
  const url = `${BASE_URL}/player/pitcherlist`;
  const res = await fetch(url, { method: "GET" });
  if (!res.ok) {
    throw new Error("Failed to fetch coach list");
  }

  const response: PitcherResponse = await res.json();
  return response;
};

페이지에서 보이시다시피 서버 컴포넌트로 생성하여 불러왔습니다. ->특징 : 콘솔에 찍히지 않고 네트워크창 깔끔 + 초기 로딩 속도 빠름
image

다들 본인이 만든 컴포넌트 서버로 돌릴 수 있으면 서버 구현해보는게 어떨까 하고 시도해보시면 좋을것같습니다.

** 디테일페이지는 현재 퍼블리싱만 되어있는 상태입니다. 추가로 해당브랜치에서 api 연결 및 구현 후 pr 올리겠습니다.**

결과물

image

image


When modifying code...

# Request Level
  - [ ] : "🔥 이대로 Merge 하면 안돼요~!"
  - [ ] : "🥹 고치면 분명 나아질 게 분명합니다.."
  - [ ] : "🤷 수정하면 좋지 않을까요?"

# Description

@hayoung78 hayoung78 added feat 새로운 기능 추가 design CSS 등 사용자 UI 변경 rename 파일 혹은 폴더명을 수정하거나 옮기는 작업만 하는 경우 labels Jan 9, 2025
@hayoung78 hayoung78 self-assigned this Jan 9, 2025
@hayoung78 hayoung78 changed the title ✚ [WV-35] feat : player - 투수 리스트 페이지 생성 ➕ [WV-35] feat : player - 투수 리스트 페이지 생성 Jan 9, 2025
@naelumcat
Copy link
Contributor

naelumcat commented Jan 9, 2025

npm run dev서버 안에서 투수 리스트 페이지 전용서버가 생기는 건가요?

image
이렇게 따로 만들어지는가 하고 궁금해서용
이건 제가 따로 찾아볼게요

@hayoung78
Copy link
Contributor Author

npm run dev서버 안에서 투수 리스트 페이지 전용서버가 생기는 건가요?

잉? 이게 뭔소리져..?!

Copy link
Member

@Jinviz Jinviz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다!

8fe7b48
…Twizviz/wizviz into WV-35-create-player-coach-page
@hayoung78 hayoung78 merged commit d69dcf7 into dev Jan 9, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design CSS 등 사용자 UI 변경 feat 새로운 기능 추가 rename 파일 혹은 폴더명을 수정하거나 옮기는 작업만 하는 경우
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants