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/#55] 회의 시간 자동 종료 기능 및 안건 시간 총합으로 회의 시간 설정 구현 #56

Merged
merged 3 commits into from
Feb 19, 2024

Conversation

Starlight258
Copy link
Collaborator

@Starlight258 Starlight258 commented Feb 19, 2024

What is this PR? 🔍

  • 회의 시간을 안건 시간의 총합으로 조정
  • 설정된 시간이 종료되면 회의가 자동으로 종료되는 기능 구현

Changes 📝

  • 회의 시간 조정: 회의 시간을 안건들의 예상 소요 시간 총합으로 설정하는 로직 추가
  • 회의 자동 종료 스케줄러 구현: 매 분마다 실행되는 스케줄러를 통해 진행 중인 회의들을 검사하고, 설정된 회의 시간이 종료되었는지 확인 후 자동으로 회의 상태를 종료로 변경

Screenshot 📷

Nan

@Starlight258 Starlight258 added this to the Sprint 3 milestone Feb 19, 2024
@Starlight258 Starlight258 self-assigned this Feb 19, 2024
@Starlight258 Starlight258 changed the title feat: 회의 시간 자동 종료 기능 및 안건 시간 총합으로 회의 시간 설정 구현 [Feat/#55] 회의 시간 자동 종료 기능 및 안건 시간 총합으로 회의 시간 설정 구현 Feb 19, 2024
@Schema(description = "안건 종류", example = "AGENDA")
private String type;

@Schema(description = "현재까지 소요된 시간", example = "00:36")
Copy link
Collaborator

Choose a reason for hiding this comment

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

✅ P3: exaple에 ss 반영이 필요해보여요 :)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

감사합니다~! 반영했습니다!!

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 👍👍

@Starlight258 Starlight258 merged commit 1261af1 into dev Feb 19, 2024
1 check failed
@Starlight258 Starlight258 deleted the feat/meeting-closed branch February 19, 2024 16:29
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