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] api 주소 수정, getDayDifference 함수 정의 #105

Merged
merged 1 commit into from
Aug 16, 2024

Conversation

darkdulgi
Copy link
Collaborator

#️⃣ 연관 이슈

📝 작업 내용

  • getDayDifference 함수 정의

두 개의 Date를 받으면 시간을 무시하고 오직 일(day) 단위의 차이를 내보냅니다. Date는 객체인지 밀리초 단위인지 상관이 없으며, @common/utils.js 파일에 정의되어 있습니다.

  • api 명세에 맞게 인터랙션 페이지를 작동하게 했습니다.

@darkdulgi darkdulgi added the feat 기능 구현 label Aug 16, 2024
@darkdulgi darkdulgi self-assigned this Aug 16, 2024
Comment on lines 44 to 46
.catch((e) => {
console.log(e);
});
Copy link
Collaborator

Choose a reason for hiding this comment

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

만약 에러가 났다면, setJoinedList를 [0,0,0,0,0] 같은걸로 설정해서 서버 오류시에도 동작은 하게 만드는 게 좋을 겁니다.


export default function joinEvent(index) {
const isLogin = userStore.getState().isLogin;
const eventDate = EVENT_START_DATE.getTime() + index * 24 * 60 * 60 * 1000;
Copy link
Collaborator

Choose a reason for hiding this comment

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

은근 24 * 60 * 60 * 1000 이라는 매직넘버가 저희 코드 지속적으로 쓰이는 것 같습니다.
하나의 상수로 뭉쳐야 할 것 같습니다...

@lybell-art lybell-art merged commit 8c24e63 into dev Aug 16, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat 기능 구현
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feat] 메인-인터랙션 모달 [feat] 메인-인터랙션 페이지
2 participants