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] Firestore에서 세션 정보를 받아오는 임시 구현을 제거하고, Remote Config에서 가져오도록 복구 #343

Merged
merged 1 commit into from
Jun 24, 2024

Conversation

jihee-dev
Copy link
Member

@jihee-dev jihee-dev commented Jun 6, 2024

Description
Firestore에서 세션 정보를 받아오는 임시 구현을 제거하고, Remote Config에서 가져오도록 복구

기존에 임시 구현으로 Firebase Firestore에서 세션을 가져올 수 있도록 되어 있었는데,
서버 도입 등으로 릴리즈 쪽에서는 Remote Config에서 받아오는 로직을 그대로 사용 중이었습니다
develop이 임시 구현으로 인해 배포할 수 없는 상황이 되어 다시 기존 로직으로 돌려두었습니다~!
release -> develop으로 머지도 함께 진행하였으니 혹시 잘못 머지된 게 있다면 요 PR에서 함께 말씀해 주시면 감사하겠습니다

임시 구현 부분(setSession 등)은 혹시 몰라서 그냥 두었는데
정리되는 대로 제거하거나 .. 완성시키면 좋을 것 같습니다!
그리고 추후에는 요런 임시 구현이 필요하거나 여러 PR로 나누어 구현할 경우
feature 브랜치를 만들어서 그쪽으로 머지하는 방향으로 진행하면 좋을 것 같습니다!

테스트 목록

  • 멤버
    • 다가오는 세션 정보
    • 세션 시작 전 10분 ~ 시작 후 10분까지 정상 출석 처리 확인
    • 세션 시작 후 10분 ~ 120분까지 지각 처리 확인
    • 세션 시작 120분 이후에는 출석 코드 입력 화면에 접근 불가
    • 점수 통계 확인
  • 어드민
    • 누적 점수 관리
    • 세션별 점수 관리

Screen Shots

Screen Shot Screen Shot

Check List

  • CI/CD 통과여부
  • Develop Mege 시 Squash and Merge 형태로 넣어주세요!
  • 1Approve 이상 Merge
  • Unit Test 작성
  • 연관된 이슈를 PR에 연결해주세요.

Copy link
Member

@toastmeister1 toastmeister1 left a comment

Choose a reason for hiding this comment

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

지희님 늦어서 미안해요 Approve 입니다~
고생하셨어요

@jihee-dev jihee-dev merged commit 5f0933d into develop Jun 24, 2024
1 check passed
@jihee-dev jihee-dev deleted the feature/fix-to-get-session-from-remote-config branch June 24, 2024 12:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants