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/#59] 회의실 나가기 기능 및 안건 수정 기능 개발, 기획 변경으로 인한 코드 수정 #63

Merged
merged 11 commits into from
Feb 21, 2024

Conversation

Starlight258
Copy link
Collaborator

@Starlight258 Starlight258 commented Feb 20, 2024

🔗 Linked Issue

resolved: #59 #60

🛠 개발 기능

  • 회의 나가기 기능 개발
  • 참가자가 없는 경우에도 조회가능하도록 개선
  • 안건 수정 기능 개발

기획 변경으로 인한 코드 수정

  • 참가자 조회 API 개선
  • 회의 스케줄링 조정: 기획 변경으로 인한 회의 시작 및 종료 스케줄링 기능 제거
  • 회의 시작 프로세스 변경

🧩 해결 방법

  • 미팅 나가기 기능: 방장이 회의를 떠날 경우 회의에 남은 사람을 랜덤으로 방장으로 설정하도록 구현했습니다.
    방장이 아닐 경우에는 회의에 떠나는 작업만 수행됩니다.

  • 안건 수정 기능 : 안건 이름과 할당 시간을 수정하는 기능을 구현했습니다.

  • 참가자 조회 기능 개선 : 참가자 조회 API에 방장 정보를 추가

  • 회의 시작 프로세스 : 안건 시작시 자동으로 회의 시작되도록 기능 구현

🔍 리뷰 포인트

  • 프로젝트 발표 기간이 다가오고있네요..! 짧은 시간에 많은 부분을 수정하다보니 PR이 커졌어요 죄송합니다! ㅠㅠ
  • 추가된 기능의 로직이 정확한지 확인 부탁드려요 !


📋 Code Review Priority Guideline

  • 🚨 P1: Request Change
    • 필수 반영: 꼭 반영해주시고, 적극적으로 고려해주세요 (수용 혹은 토론).
  • 💬 P2: Comment
    • 권장 반영: 웬만하면 반영해주세요.
  • 👍 P3: Approve
    • 선택 반영: 반영해도 좋고 넘어가도 좋습니다. 그냥 사소한 의견입니다.

@Starlight258 Starlight258 added this to the Sprint 4 milestone Feb 20, 2024
@Starlight258 Starlight258 self-assigned this Feb 20, 2024
@Starlight258 Starlight258 changed the title Feat/meeting agenda updates [Feat/#59] 회의실 나가기 기능 및 안건 수정 기능 개발 Feb 20, 2024
@Starlight258 Starlight258 changed the title [Feat/#59] 회의실 나가기 기능 및 안건 수정 기능 개발 [Feat/#59] 회의실 나가기 기능 및 안건 수정 기능 개발, 기획 변경으로 인한 코드 수정 Feb 20, 2024
@FacerAin FacerAin self-requested a review February 20, 2024 16:19
Copy link
Collaborator

@FacerAin FacerAin left a comment

Choose a reason for hiding this comment

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

LGTM :)


public AgendaPatchResponse patchAgenda(Long meetingId, Long agendaId, AgendaPatchRequest patchRequest) {
// 회의 존재 여부만 확인
boolean meetingExists = meetingRepository.existsById(meetingId);
Copy link
Collaborator

Choose a reason for hiding this comment

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

existsById 활용하여 컬럼 존재 여부 확인 좋습니다. 👍

@Starlight258 Starlight258 merged commit ef7d32a into dev Feb 21, 2024
1 check passed
@Starlight258 Starlight258 deleted the feat/meeting-agenda-updates branch March 18, 2024 00:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants