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

Feature/세미나 qa 수정사항 #381

Merged
merged 7 commits into from
Oct 8, 2023

Conversation

shkisme
Copy link
Contributor

@shkisme shkisme commented Oct 7, 2023

🔥 Related Issue

close: #379
close: #380
issue: #239

📝 Description

  • 세미나 생성 시 회원의 출석 전 데이터의 attendTime을 날짜에 맞게 넣어주었습니다.
    • 넣어주지 않았더니 DB에서 현재 시간으로 설정이 되어서 미래의 세미나를 생성하게 된다면 문제가 되었습니다.
  • 가장 최근에 마감된 세미나 조회 api를 지난 세미나(~어제) 중 openTime을 기준으로 가장 현재와 가까운 세미나 하나를 조회하도록 변경하였습니다.
    • 키퍼 세미나는 현재 하루에 하나만 생성되게끔 구현되어 있어서, 마감된이라는 조건이 의미없는 것 같아 제외하게 되었습니다. (UI 상으로도 지각, 출석 마감 시간이 최대 15분이라 그렇기도 합니다.)
  • 세미나 생성 시 정회원이 아니었으나, 현재 정회원인 회원의 출석 처리
    • 세미나 생성 당시 정회원이 아니었어도, 세미나 출석 시점으로 정회원이라면 출석이 되도록 코드를 수정하였습니다.
    • 세미나 생성 당시 정회원 아니여서 출석 전 데이터가 없고 + 출석 api 자체를 호출하지 않았을 경우 결석으로 처리는 TODO 입니다!
  • 출석 전, 출석, 결석 상태로 변경할 때 기존의 사유를 지우도록 수정하였습니다.

⭐️ Review Request

배포가...! 얼마 남지 않았습니다...!!

@shkisme shkisme added 🐞 bug 예기치 않은 문제 또는 의도하지 않은 동작 ⭐️ backend 백엔드 코드 작성 labels Oct 7, 2023
@shkisme shkisme requested review from gusah009 and 02ggang9 October 7, 2023 12:18
@shkisme shkisme self-assigned this Oct 7, 2023
Copy link
Collaborator

@02ggang9 02ggang9 left a comment

Choose a reason for hiding this comment

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

기대가 됩니다..!! 수고하셨습니다~!

@shkisme shkisme merged commit 5e94450 into develop Oct 8, 2023
1 check passed
@shkisme shkisme deleted the feature/#379-세미나-QA-수정사항 branch October 8, 2023 01:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⭐️ backend 백엔드 코드 작성 🐞 bug 예기치 않은 문제 또는 의도하지 않은 동작
Projects
None yet
2 participants