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] 어드민 오류 수정 #185

Merged
merged 8 commits into from
Aug 22, 2024
Merged

[Fix] 어드민 오류 수정 #185

merged 8 commits into from
Aug 22, 2024

Conversation

jhj2713
Copy link
Member

@jhj2713 jhj2713 commented Aug 21, 2024

🖥️ Preview

close #184

✏️ 한 일

  • 이벤트 날짜/시간 설정 시 초가 들어왔을 때 오류 수정
  • 추첨 이벤트 다시 추첨 시 오류 수정
  • 검색 시 enter 처리 (submit)

❗️ 발생한 이슈 (해결 방안)

이미 추첨한 이력이 있는 경우 추첨 API를 호출하면 오류가 발생합니다. 오류를 없애기 위해서 추첨한 이력이 있는 경우 이전 추첨 이력을 삭제하고 다시 추첨 API를 호출하도록 수정했습니다.

❓ 논의가 필요한 사항

@jhj2713 jhj2713 added the fix 버그가 발생 label Aug 21, 2024
@jhj2713 jhj2713 requested a review from sooyeoniya August 21, 2024 04:23
@jhj2713 jhj2713 self-assigned this Aug 21, 2024
Copy link

빌드를 성공했습니다! 🎉

Copy link

빌드를 성공했습니다! 🎉

Copy link

빌드를 성공했습니다! 🎉

Copy link

빌드를 성공했습니다! 🎉

Copy link
Member

@sooyeoniya sooyeoniya left a comment

Choose a reason for hiding this comment

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

귀찮은 오류 잘 잡으셨네용 ㅎㅎ 수고많으셨어요 :)

@sooyeoniya sooyeoniya merged commit 3dba62d into dev Aug 22, 2024
2 checks passed
@jhj2713 jhj2713 deleted the fix/#184-admin-lottery branch August 23, 2024 08:10
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