Skip to content

Fix : 여러 기수 소속 회원 명단 누락 수정 - #570

Merged
uykm merged 2 commits into
developfrom
fix/member/#569-multi-cohort-membership
Sep 30, 2026
Merged

uykm merged 2 commits into
developfrom
fix/member/#569-multi-cohort-membership

Conversation

@uykm

@uykm uykm commented Sep 29, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Tasks

  • 여러 기수에 소속된 회원을 각 기수 명단에 모두 포함하도록 변경
    • findAllByCohort, findAllByCohortId, findAllMemberIdsByCohortIdAndAuthorityId를 마지막 소속 행이 아닌 해당 기수 소속 여부로 조회
    • 출석부 생성, 공지 읽음·푸시, 리마인더, 회식 초대, 과제 제출 행 생성에 반영
  • 대표 기수(회원 목록 기수 표시, 내 정보 cohort)를 입력 순서가 아닌 가장 높은 기수 기준으로 변경
  • 기수 init API(POST /v1/members/authority/cohort/init/{cohortId}/{memberId})에 update:member 권한 체크 추가
  • MemberLatestCohortTest 추가

ETC

Screenshot

uykm and others added 2 commits September 29, 2026 17:27
- 기수별 명단 조회(findAllByCohort, findAllByCohortId, findAllMemberIdsByCohortIdAndAuthorityId)를
  최신 소속 행 하나가 아닌 해당 기수 소속 여부로 판단
- 대표 기수(회원 목록 표시, 내 정보 cohort)는 입력 순서가 아닌 가장 높은 기수 기준으로 선택

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01WcGtHShfT1PVH4iSiufgaz
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01WcGtHShfT1PVH4iSiufgaz
@uykm uykm added the 🐞 Bug label Sep 29, 2026
@uykm uykm self-assigned this Sep 29, 2026
@coderabbitai

coderabbitai Bot commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Advanced

Run ID: 05203f5b-aa8d-4456-86bf-39a74aca6e9b


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@uykm uykm changed the title Fix : 여러 기수 소속 회원을 각 기수 명단에 모두 포함 및 기수 init API 권한 체크 Fix : 여러 기수 소속 회원 명단 누락 수정 Sep 29, 2026
@uykm
uykm requested review from hwistlezz and removed request for heeun98 September 29, 2026 08:30
Base automatically changed from refactor/cohort-role-system to develop September 29, 2026 13:15
@uykm
uykm merged commit 36cf62f into develop Sep 30, 2026
1 check passed
@uykm
uykm deleted the fix/member/#569-multi-cohort-membership branch September 30, 2026 13:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant