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-40] refactor : main page refactor #53

Merged
merged 18 commits into from
Jan 16, 2025
Merged

Conversation

hee2323
Copy link
Contributor

@hee2323 hee2323 commented Jan 16, 2025

개요

메인페이지 리팩터링 했습니다


PR Checklist

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

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

PR details

  • 경기 일정 api 연결, 스켈레톤 생성 및 적용
  • 팀 랭킹 배너 생성, api 연결, 스켈레톤 생성 및 적용
  • 주차 예약 배너 생성, 클릭 시 주차 예약 페이지 이동 추가
  • 갤러리 하단 갤러리 페이지 이동 버튼 삭제
  • 온라인 스토어, 이달의 선수 배너 클릭 시 페이지 이동 추가

When modifying code...

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

# Description

@hee2323 hee2323 added feat 새로운 기능 추가 design CSS 등 사용자 UI 변경 refactor 작업수정, 코드 유지보수 remove 파일, 폴더 삭제 labels Jan 16, 2025
@hee2323 hee2323 self-assigned this Jan 16, 2025
@hee2323 hee2323 changed the title + [WV-40] refactor : main page refactor ➕ [WV-40] refactor : main page refactor Jan 16, 2025
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.

수고하셨습니다!

Copy link
Contributor

@hayoung78 hayoung78 left a comment

Choose a reason for hiding this comment

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

확인했습니다 근데 pr에 ui가 없쉉!!!!

api/main/apis.ts Outdated
Comment on lines 8 to 11
headers: {
"Content-Type": "application/json",
},
});
Copy link
Contributor

Choose a reason for hiding this comment

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

이거 디폴트값인걸로 알고있는데 빼면 좋지않을 까욤~~~

Copy link
Contributor Author

Choose a reason for hiding this comment

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

제거했습니다

}}
/>
<div className="absolute w-[540px] h-[190px] inset-0 px-10 pt-4 flex flex-col justify-center mr-10">
<p className="text-l text-red-500 font-medium mb-2">
Copy link
Contributor

Choose a reason for hiding this comment

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

red-500?!?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

수정했습니다

@hee2323 hee2323 merged commit cbce68a into dev Jan 16, 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 새로운 기능 추가 refactor 작업수정, 코드 유지보수 remove 파일, 폴더 삭제
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants