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

[refactor] 코드 리팩터링 #106

Merged
merged 2 commits into from
Aug 17, 2024
Merged

[refactor] 코드 리팩터링 #106

merged 2 commits into from
Aug 17, 2024

Conversation

darkdulgi
Copy link
Collaborator

📝 작업 내용

  • DAY_MILLISEC 상수 정의

@common/constants.js 파일에 있으며 하루를 밀리초 단위로 나타낸 값입니다. (24 * 60 * 60 * 1000)

  • prettier 설정 약간 수정

공간이 남아도는데 불필요한 줄바꿈이 많은 것 같아 printWidth를 100으로 했습니다. (default는 80)

  • 인터랙션 페이지 리팩터링

사용자의 참여 여부를 나타내는 리스트 상태인 joinedList를 true, false 두 값만 갖게 하고, 아직 열리지 않은 이벤트에 대한 비활성화 처리는 각 컴포넌트에서 날짜를 계산해 하도록 했습니다. 따라서 이름도 isJoinedList로 바꿨습니다.

@darkdulgi darkdulgi added chore 개발과 직접적인 연관이 없음 refactor 코드 리팩토링 labels Aug 17, 2024
@darkdulgi darkdulgi self-assigned this Aug 17, 2024
@@ -5,3 +5,5 @@ export const EVENT_START_DATE = new Date(2024, 8, 9);

export const SERVICE_TOKEN_ID = "AWESOME_ORANGE_ACCESS_TOKEN";
export const ADMIN_TOKEN_ID = "AWESOME_ORANGE_ADMIN_ACCESS_TOKEN";

export const DAY_MILLISEC = 1000 * 24 * 60 * 60;
Copy link
Collaborator

Choose a reason for hiding this comment

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

상수화 좋습니다:)

@lybell-art lybell-art merged commit d1b8b5c into dev Aug 17, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore 개발과 직접적인 연관이 없음 refactor 코드 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants