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

feat(time): 모의 수강신청 페이지 개발 #263

Merged
merged 43 commits into from
Feb 10, 2025
Merged

feat(time): 모의 수강신청 페이지 개발 #263

merged 43 commits into from
Feb 10, 2025

Conversation

yummjin
Copy link
Contributor

@yummjin yummjin commented Jan 15, 2025

Summary

related: #256

모의 수강신청 페이지의 기본적인 타입 설정을 하고, 레이아웃을 만들었습니다.

Tasks

  • time 작업환경에서 모의 수강신청 페이지로 이어지는 디렉토리 Sugang을 추가했어요.
  • 경기플러스 화면에서 수강신청 페이지 바로가기 버튼을 만들었습니다.
  • 수강신청 페이지를 최대한 본떠 UI를 구성했어요.
  • 소망가방을 사용한 수강신청부터 구현하기로 정했고, 나머지 탭 버튼은 비활성화했어요.
  • 더미 데이터를 사용하기 위한 Data 디렉토리를 생성하고, 내부에 더미 데이터를 넣었습니다.
  • 더미 데이터는 ChatGPT를 사용해 생성했어요.
  • 모의 수강신청 화면에서 신청, 삭제에 해당하는 부분은 차후에 버튼이 들어가게 돼요. 이후에 작업할 예정입니다.

To Reviewer

  • 파일 구조와 스타일링 등의 부분에서, 개선이 필요한 부분이 보인다면 알려주세요.
  • 더 나은 타입 선언이나 상수 핸들링 방법이 있다면 조언 부탁드립니다.

Screenshot

image

@yummjin yummjin added ✨ Feature 새로운 기능 명세 및 개발 ⏰ Time time 프로젝트 관련 labels Jan 15, 2025
@yummjin yummjin self-assigned this Jan 15, 2025
@yummjin yummjin requested a review from Jeong-Ag as a code owner January 15, 2025 06:00
Copy link

changeset-bot bot commented Jan 15, 2025

🦋 Changeset detected

Latest commit: 0e114aa

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 6 packages
Name Type
@clab-platforms/time Minor
@clab-platforms/icon Minor
@clab-platforms/land Patch
@clab-platforms/member Patch
@clab-platforms/design-system Patch
@clab-platforms/auth Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@yummjin yummjin requested review from gwansikk and SWARVY January 15, 2025 06:00
@gwansikk
Copy link
Member

gwansikk commented Jan 15, 2025

@cho4u4o pnpm-lock.yaml 충돌을 해결해주세요~

@gwansikk
Copy link
Member

@yummjin yummjin linked an issue Jan 15, 2025 that may be closed by this pull request
3 tasks
@yummjin
Copy link
Contributor Author

yummjin commented Jan 15, 2025

PR 본문에 키워드로 이슈를 연결해주세요

ref:

#256 이슈를 PR에 연결하면 되나요?

@gwansikk
Copy link
Member

PR 본문에 키워드로 이슈를 연결해주세요
ref:

#256 이슈를 PR에 연결하면 되나요?

이슈를 닫으려면 closed:, PR을 다르게 하여 이어서 하려면 related: 키워드를 달아주세요

Copy link
Member

@gwansikk gwansikk left a comment

Choose a reason for hiding this comment

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

수정하여 다시 리뷰를 받고자 하면 오른쪽 상단의 리뷰 재요청을 눌러주세요

@yummjin yummjin requested a review from gwansikk January 16, 2025 04:44
Copy link
Contributor

@SWARVY SWARVY 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
Member

Choose a reason for hiding this comment

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

"@types/react": "^18",
"@types/react-dom": "^18",

리액트 타입도 19 버전 대응해주세요

Copy link
Contributor

@SWARVY SWARVY left a comment

Choose a reason for hiding this comment

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

위에꺼 수정한다음 머지해주시면 될 것 같습니다 :)

@yummjin yummjin merged commit 6a0ffe4 into main Feb 10, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature 새로운 기능 명세 및 개발 ⏰ Time time 프로젝트 관련
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants