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

[fix] 잡다하게 수정 #130

Merged
merged 4 commits into from
Aug 22, 2024
Merged

[fix] 잡다하게 수정 #130

merged 4 commits into from
Aug 22, 2024

Conversation

darkdulgi
Copy link
Collaborator

📝 작업 내용

  • 사용자 정보 등록 모달에서 이미 존재하는 전화번호로 인증을 시도할 때 나타나는 문구를 하단의 링크를 클릭하라고 수정
  • 번호 인증 요청 API 인자 추가

백엔드의 요청대로 /api/v1/event-user/send-auth 뒤에 이벤트 프레임 ID를 붙입니다.

  • 어드민 기대평 조회에서 사용자들이 작성한 기대평의 작성시간을 한국 현지시간에 맞게 표시
  • 어드민 기대평 페이지에서 이벤트 검색할 때 포맷에 맞는 id의 이벤트들만 자동검색에 뜨게 함

HD_012345_678 형식 이외의 포맷은 비정상적인 id라고 판단, 자동검색에서 무시됩니다.

  • 어드민 선착순 이벤트 생성/수정 시 현재 날짜와 같거나 이미 지난 날짜의 이벤트로 생성/수정하는 것을 막음

@darkdulgi darkdulgi added the fix 버그 및 오류 수정 label Aug 22, 2024
@darkdulgi darkdulgi self-assigned this Aug 22, 2024
Copy link
Collaborator

@lybell-art lybell-art left a comment

Choose a reason for hiding this comment

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

수고하셨습니다.

method: "post",
body,
});
return "";
} catch (e) {
return handleError({
400: "잘못된 요청 형식입니다.",
409: "등록된 참여자 정보가 있습니다.",
409: "이미 등록된 전화번호가 존재합니다. 하단의 '이미 정보를 입력하신 적이 있으신가요?'를 클릭하세요.",
Copy link
Collaborator

Choose a reason for hiding this comment

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

친절해졌군요

Comment on lines -30 to -43
function getDate(createdAt) {
const yy = createdAt.slice(2, 4);
const mm = createdAt.slice(5, 7);
const dd = createdAt.slice(8, 10);
return `${yy}-${mm}-${dd}`;
}

function getTime(createdAt) {
const hh = createdAt.slice(11, 13);
const mm = createdAt.slice(14, 16);
const ss = createdAt.slice(17, 19);
return `${hh}:${mm}:${ss}`;
}

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 1924be5 into dev Aug 22, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix 버그 및 오류 수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants