[Change] : Member
에 포함했던 출석 관련 기능을 Attendance
로 이동
#48
goldentrash
started this conversation in
Ideas
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
상황 설명
현재
MemberController
에는GET /api/v1/members/{batch}/attendances?year&month
로 모든 회원의 출석 정보를 조회하는 API와PATCH /api/v1/members/{batch}/attendances?year&month
로 모든 회원의 출석 정보를 수정하는 API가 포함되어 있습니다.이 두 API는 "회원"과 관련된 기능이라기보다는 "출석"과 관련된 기능으로 보여집니다. (제가 설계했는데 다시 보니 너무한 느낌이네요)
관련 비즈니스 로직(
MemberService.getMemberAttendanceWithBatchAndPeriod
,MemberService.updateAttendances
)을 확인해 보니 모두attendanceRepository
와participantRepository
에 직접적으로 접근하는 형태입니다.구체 제안
GET /api/v1/members/{batch}/attendances?year&month
=>GET /api/v1/attendances/participants?year&month
PATCH /api/v1/members/{batch}/attendances?year&month
=>PATCH /api/v1/attendances/participants?year&month
MemberService.getMemberAttendanceWithBatchAndPeriod
=>ParticipantService.getAllMonth
MemberService.updateAttendances
=>ParticipantService.updateAttendances
여기서부터의 내용은 2차 개발 변경 사항을 기반으로 합니다. 10/25/2024 시점 운영진 간 논의는 완료되었지만, 아직 모든 개발진과 논의를 하지 않았기에 추후 변경될 가능성이 있습니다.
상황 설명
변경 (예정) 사항 : "모든 출석은 해당 기수 전원을 대상으로 한다."
(이전에는 출석 대상에 대해 외부인까지도 확장을 고려, 현재
기수
를 입력 받아 해당하는 인원을 대상으로 설정하도록 구현한 상황)사실, 위에서 이미
getMemberAttendanceWithBatchAndPeriod
를getAllMonth
로 변경하는 등기수
를 요청 명세에서 제외하는 방향으로 제안 한 내용이 있습니다. 이 제안 또한 출석 대상을 현 기수 모든 회원으로 변경하였기 때문에 굳이 기수를 입력으로 받을 필요가 없어진 사항을 반영한 것입니다. (출석 날짜를 기준으로 기수 판단 가능)구체 제안
AttendanceRepository.findAllAttendanceInfoByBatchAndPeriod
=>AttendanceRepository.findAllAttendanceInfoByPeriod
ParticipantRepository.findAllByBatchAndStartAtBetween
=>ParticipantRepository.findAllByPeriod
Beta Was this translation helpful? Give feedback.
All reactions