Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

merge develop <- feature/lecture-domain #42

Merged
merged 11 commits into from
Jan 22, 2025
Merged

Conversation

Hm-source
Copy link
Contributor

@Hm-source Hm-source commented Jan 21, 2025

Summary

#41

강의 업로드, 삭제, 조회 기능 구현.

Tasks

  • 강의 API
    • POST /course/{courseId}/lecture: 강의 생성 API 구현 (동영상 파일 업로드 포함)
    • DELETE /course/{courseId}/lecture/{lectureId}: 강의 삭제 API 구현
    • GET /course/{courseId}/lecture: 특정 과정의 강의 목록 조회 API 구현
    • GET /course/{courseId}/lecture/{lectureId}: 특정 과정의 특정 강의 조회 API 구현
    • yml S3 설정 추가 및 aws S3 사용을 위한 의존성 추가
    • 권한 검증 로직 작성 (현재 로그인 확인 및 과정과 연관되어 있는지 확인)
    • swagger 작성
    • swagger 테스트 완료 및 postman 테스트 완료(파일 업로드 API만)

To Reviewer

Screenshot

  • 강의 생성 api 호출 시 S3 등록
    스크린샷 2025-01-22 오전 9 35 39
  • 스웨거에서 동작 안해서, postman으로 대체 테스트 완료.
스크린샷 2025-01-22 오전 12 53 43

스크린샷 2025-01-22 오전 12 53 55

  • 삭제
스크린샷 2025-01-22 오전 1 33 37
  • 조회
스크린샷 2025-01-22 오전 1 33 13 스크린샷 2025-01-22 오전 1 32 59

Copy link
Contributor

@LeeShinHaeng LeeShinHaeng left a comment

Choose a reason for hiding this comment

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

LGTM! 👍

@Hm-source Hm-source merged commit d90de9d into develop Jan 22, 2025
1 check passed
@LeeShinHaeng LeeShinHaeng deleted the feature/lecture-domain branch January 24, 2025 02:48
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants