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.

Sign up for GitHub

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

Jump to bottom

Dev -> main 배포 #56

Merged
merged 62 commits into from
Jan 16, 2025
Merged

Dev -> main 배포 #56

merged 62 commits into from
Jan 16, 2025

Conversation

hee2323
Copy link
Contributor

@hee2323 hee2323 commented Jan 16, 2025

개요

  • main 페이지 리팩터링(api연결 및 배너별 컴포넌트 생성)
  • 엔드포인트 수정
  • 카카오맵 로드 문제 수정
  • 뉴스 디테일 페이지 추가
  • 투수 페이지 추가
  • 웰페이퍼 페이지 추가

PR Checklist

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

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

PR details


When modifying code...

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

# Description

hayoung78 and others added 30 commits January 10, 2025 17:32
@hayoung78
♻️ WV-42 refactor : 중복된 타입 정리
ff72752
@hayoung78
➕ WV-42 : pitcherDetail api 함수 생성
0e2aac5
@hayoung78
♻️ WV-42 refactor : playerCard에 pathname props 추가
aa13f56
@hayoung78
➕ WV-42 feat : pitcherDetail page연결
5249c1b
@hayoung78
♻️ WV-42 refactor : 폴더구조 변경
c395e60
@hayoung78
➕ WV-42 feat : pitcher detail 컴포넌트 생성
d0eafbb
@naelumcat
💄[WV-44] design : create-bi-page 추가
74ef8a2
@hee2323
♻️ WV-40 refactor : api 이름 수정
a73bd4b
@hayoung78
➕ WV-45 feat : news-detail page 생성중
2085ce3
@hayoung78
➕ WV-45 feat : news-detail api 함수 생성
42ac9b0
@Jinviz
🔧 WV-46 fix : 카카오SDK 경로 https로 변경
516fd7d
@naelumcat
♻️ WV-44 refactor : iframe 불필요한 속성 제거
e17dd06
@Jinviz
🔧 WV-46 fix : 백엔드 요청 프록시 URL로 설정
be33f3e
@Jinviz
🔧 WV-37 fix : 카카오맵 로드 문제 해결
c1b9c18
@hayoung78
➕ WV-45 feat : news-detail page 생성
111ed49
@hayoung78
➕ WV-45 feat : news-detail page 생성
884f8ed
@Jinviz
🔧 WV-37 fix : body 태그 안으로 스크립트 로드 위치
e050a73
@naelumcat
💄[WV-43] design : create-wallpaper-page 추가
12a0b3d
@hee2323
💄 WV-40 design : 메인 페이지 배너 이미지 추가
50b27d7
@hee2323
💄 WV-40 design : 기존 대체하던 캡처 이미지 제거
20c2cb5
@hee2323
♻️ WV-40 refactor : api 호출하는 react query 작성 시 queryparams로 묶어서 관심사 분리
a32c550
@hee2323
+ WV-40 feat : 경기 일정, 랭킹 배너 스켈레톤 생성
0a7ef0a
@hee2323
+ WV-40 feat : 팀 랭킹, 주차 예약 배너 생성
223cfa3
@hee2323
+ WV-40 feat : 팀 랭킹 api & type 추가
c46da47
@Jinviz
🔧 WV-46 fix : API 환경 변수에 맞추어 경로 수정
2d76428
@hee2323
♻️ WV-40 refactor : boxscore api 호출 시 기본 queryParams값 추가
e068707
@hee2323
♻️ WV-40 refactor : 경기 일정 boxscore api 적용, 팀 랭킹, 주차 예약 배너 컴포넌트 적용
a7d8613
@hee2323
🔥 WV-40 remove : 경기일정 상수값 제거
d5dc907
@hee2323
♻️ WV-40 refactor : 비디오 페이지 없어짐에 따라 불필요한 버튼 제거
817fca5
@hee2323
♻️ WV-40 refactor : 갤러리 하단 스토어 배너, 이달의 선수 배너 클릭 시 페이지 이동
25e502c
Jinviz
Jinviz approved these changes Jan 16, 2025
View reviewed changes
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.

✈️

naelumcat
naelumcat approved these changes Jan 16, 2025
View reviewed changes
Copy link
Contributor

@naelumcat naelumcat left a comment

Choose a reason for hiding this comment

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

확인했습니다

@hayoung78 hayoung78 added feat 새로운 기능 추가 design CSS 등 사용자 UI 변경 fix 버그수정 refactor 작업수정, 코드 유지보수 remove 파일, 폴더 삭제 rename 파일 혹은 폴더명을 수정하거나 옮기는 작업만 하는 경우 labels Jan 16, 2025
@hayoung78 hayoung78 merged commit fa44cf0 into main Jan 16, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@naelumcat naelumcat naelumcat approved these changes

@hayoung78 hayoung78 hayoung78 approved these changes

@Jinviz Jinviz Jinviz approved these changes

Assignees

@Jinviz Jinviz

@hee2323 hee2323

@hayoung78 hayoung78

@naelumcat naelumcat

Labels
design CSS 등 사용자 UI 변경 feat 새로운 기능 추가 fix 버그수정 refactor 작업수정, 코드 유지보수 remove 파일, 폴더 삭제 rename 파일 혹은 폴더명을 수정하거나 옮기는 작업만 하는 경우
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@hee2323 @Jinviz @hayoung78 @naelumcat