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 추가 #89

Merged
merged 14 commits into from
Feb 1, 2025

Conversation

Moveuk
Copy link
Member

@Moveuk Moveuk commented Jan 31, 2025

📌 관련 이슈

✨ PR 내용

시작할 때의 생각보다 필요한 로직이 많아서 많이 두꺼워 진 것 같네요. 화면단위로 나눌 걸 그랬나 싶기도하네요 ㅠㅠ

  • 운영중인 스터디 목록 추가

  • 스터디 전체의 출석율, 인증율 계산

  • 스터디원 각각의 출석율, 인증율 계산

  • 스터디원이 모든 인증을 확인 받았는 지 확인

  • Study 컬럼 추가 : kakaoUrl - figma에 있어 추가하였습니다.

  • 기존 mypage 로직 일부 수정

📚 레퍼런스 혹은 궁금한 사항들

  • 테스트를 진행하였으나 탈퇴, 승인 탈퇴 등의 변수가 많아서 놓친 부분이 있는지 추후에 한번 더 확인해야 안전할 것 같습니다!

📸 스크린샷(선택)

@Moveuk Moveuk added the enhancement New feature or request label Jan 31, 2025
@Moveuk Moveuk requested a review from uuujini January 31, 2025 17:55
@Moveuk Moveuk self-assigned this Jan 31, 2025
@Moveuk Moveuk changed the base branch from main to develop January 31, 2025 17:56
uuujini
uuujini previously approved these changes Feb 1, 2025
Copy link
Member

@uuujini uuujini left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!!!
코멘트 하나만 확인 부탁드립니다!

@Moveuk Moveuk merged commit 9b5a9e7 into develop Feb 1, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants