Skip to content

refactor: 커맨드에 따른 권한 검증 로직 반영 #22

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

Merged

Conversation

TaeyeonRoyce
Copy link
Contributor

@TaeyeonRoyce TaeyeonRoyce commented Mar 19, 2025

작업 내용

  • 각 명령어에서 허용된 권한에 따라 검증이 이루어지도록 로직을 수정 하였습니다.

관련 이슈

PR 체크리스트

  • 테스트는 모두 통과했나요?
  • 빌드는 성공했나요?
  • 코드 포맷팅을 진행했나요?
  • PR 내부의 예시는 삭제하셨나요?

@TaeyeonRoyce TaeyeonRoyce self-assigned this Mar 19, 2025
@TaeyeonRoyce TaeyeonRoyce added the refactor Code refactoring, dependency updates, and other housekeeping activities label Mar 19, 2025
@TaeyeonRoyce TaeyeonRoyce requested review from haeyoon1 and davidolleh and removed request for haeyoon1 March 19, 2025 07:20
Copy link
Collaborator

@davidolleh davidolleh left a comment

Choose a reason for hiding this comment

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

수고하셨습니다! 👍

@@ -76,12 +78,17 @@ public LocalDateTime parseScheduledTime(String timeString) {
}
}

public void IsValidscheduledTime(LocalDateTime time) {
public void IsValidScheduledTime(LocalDateTime time) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

혹시 함수 이름이 대문자로 시작하는데 소문자로 변경 부탁드립니다.

Copy link
Member

@haeyoon1 haeyoon1 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

@TaeyeonRoyce TaeyeonRoyce merged commit 7815fbe into main Mar 19, 2025
@TaeyeonRoyce TaeyeonRoyce deleted the 21-refactor-커맨드에-따른-권한-검증-로직-반영 branch March 19, 2025 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor Code refactoring, dependency updates, and other housekeeping activities
Projects
None yet
Development

Successfully merging this pull request may close these issues.

refactor: 커맨드에 따른 권한 검증 로직 반영
3 participants