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: 개인정보 수집 안내 페이지 상수 데이터 업데이트 #170

Merged
merged 2 commits into from
Aug 29, 2024

Conversation

geongyu09
Copy link
Collaborator

주요 변경사항

개인 정보 수집 안내 페이지 내용을 업데이트합니다.

  • 페이지에서 사용하는 상수 데이터를 이번 신입모집 일정에 맞추어 업데이트 하였습니다.
  • 페이지에서 최신 기수의 상수 데이터를 가져올 수 있도록 동적 import 로 변경하였습니다.
  • 추가적으로 이름이 모호한 느낌이 있어 jsDoc으로 해당 상수에 대해서 설명을 추가하였습니다.

리뷰어에게...

  • 큰 변화는 없습니다..! 더 좋은 개선 사항이 있을지 궁금 하기도 하네요!
  • 상수 데이터에 설명을 추가한게 조금 어색해 보이기도 하네요. 굳이 라는 생각이 들기도 합니다. 이와 관련해서 의견 부탁드립니다!

관련 이슈

closes #169

@geongyu09 geongyu09 requested review from 2yunseong and smb0123 August 24, 2024 12:01
@geongyu09 geongyu09 self-assigned this Aug 24, 2024
Copy link
Collaborator

@2yunseong 2yunseong left a comment

Choose a reason for hiding this comment

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

너무 좋습니다 👍 👍 동적 import 방식도 프로젝트에 맞는 해결방식 같아 재밌네요 :)

상수 데이터에 설명을 추가한게 조금 어색해 보이기도 하네요. 굳이 라는 생각이 들기도 합니다. 이와 관련해서 의견 부탁드립니다!

저는 좋은 주석이라고 생각합니다~! 고생하셨습니다 💯

Copy link
Collaborator

@smb0123 smb0123 left a comment

Choose a reason for hiding this comment

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

import 구문을 동적으로 받아오는 방식은 처음 보는데 되게 좋네요 !
저도 상수 데이터가 이름이 애매하다고 생각이 드네요. 좋은 것 같아요 !

@geongyu09 geongyu09 merged commit 1b96ff7 into main Aug 29, 2024
1 check passed
@2yunseong 2yunseong deleted the feat/169-personal-info-update branch September 9, 2024 06:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FE] feat: 개인 정보 수집 안내 페이지 관련 날짜 업데이트
3 participants