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.

Sign up for GitHub

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

Jump to bottom

[BE] 가능 시간 추가 API 구현 #20

Merged
merged 6 commits into from
Jul 16, 2024
Merged

[BE] 가능 시간 추가 API 구현 #20

merged 6 commits into from
Jul 16, 2024

Conversation

seokmyungham
Copy link
Contributor

@seokmyungham seokmyungham commented Jul 16, 2024 •
edited
Loading

관련 이슈

작업 내용

  • 원시 값 포장
  • 가능 시간을 생성하는데 필요한 검증 로직 추가
  • 가능 시간을 저장

특이 사항

  • 현재 엔티티 구조 설계와 ERD가 미흡하여 추후 개선되어야 할 필요가 있습니다.
  • 미니 해커톤 24시간 내에 구현하였습니다.
  • 가장 작은 단위의 핵심 기능을 빠르게 구현한 초기 프로토타입 입니다.

리뷰 요구사항 (선택)

@seokmyungham seokmyungham added 🐈‍⬛ 백엔드 백엔드 관련 이슈에요 :) 🚀 기능 기능을 개발해요 :) labels Jul 16, 2024
@seokmyungham seokmyungham added this to the 2차 데모데이 milestone Jul 16, 2024
ikjo39
ikjo39 reviewed Jul 16, 2024
View reviewed changes
Copy link
Contributor

@ikjo39 ikjo39 left a comment

Choose a reason for hiding this comment

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

수정할 리뷰사항이 몇가지 있어 요청 드립니다!

backend/src/main/java/com/woowacourse/momo/controller/schedule/ScheduleController.java Outdated Show resolved Hide resolved
backend/src/main/java/com/woowacourse/momo/service/schedule/ScheduleService.java Outdated Show resolved Hide resolved
ehBeak
ehBeak approved these changes Jul 16, 2024
View reviewed changes
Copy link
Contributor

@ehBeak ehBeak left a comment

Choose a reason for hiding this comment

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

👍

@seokmyungham seokmyungham merged commit c814d09 into develop Jul 16, 2024
hw0603 referenced this pull request in hw0603/momo-cicd-playground Jul 19, 2024
@seokmyungham @hw0603
[BE] 가능 시간 추가 API 구현 (#20)
5c36d92 
* feat(Guest): 문자열 Name 원시값 포장 객체 생성

* feat(ScheduleService): 가능 시간을 저장하는 기능 메서드 추가

* feat(ScheduleController): 가능 시간을 저장하는 매핑 핸들러 추가

* fix(ScheduleService): 해당 약속에 대해 이미 선택했던 시간들을 삭제하도록 코드 수정

* refactor: AllArgsConstructor 를 RequiredArgsConstructor 어노테이션으로 변경

* refactor(GuestName): 기본 생성자 접근 제어자를 protected로 변경
@ikjo39 ikjo39 deleted the feat/15-add-schedule branch July 21, 2024 11:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@seunghye218 seunghye218 seunghye218 approved these changes

@ehBeak ehBeak ehBeak approved these changes

@ikjo39 ikjo39 ikjo39 approved these changes

@hw0603 hw0603 hw0603 approved these changes

Assignees

@ehBeak ehBeak

@seunghye218 seunghye218

@seokmyungham seokmyungham

Labels
🐈‍⬛ 백엔드 백엔드 관련 이슈에요 :) 🚀 기능 기능을 개발해요 :)
Projects
2024-momo
Status: Done
Milestone
2차 데모데이
Development

Successfully merging this pull request may close these issues.

[BE] 가능 시간 추가 API를 구현해요 :)
5 participants
@seokmyungham @hw0603 @ikjo39 @ehBeak @seunghye218