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] Video 관련 API 개발 #51

Merged
merged 55 commits into from
Aug 9, 2024
Merged

[FEAT] Video 관련 API 개발 #51

merged 55 commits into from
Aug 9, 2024

Conversation

2tle
Copy link
Contributor

@2tle 2tle commented Aug 7, 2024

작성자: @2tle

close #8

체크 리스트

  • 적절한 제목으로 수정했나요?
  • 상단에 이슈 번호를 기입했나요?
  • Target Branch를 올바르게 설정했나요?
  • Reviewers/Assignees/Labels을 알맞게 설정했나요?

작업 내역

  • 대담 영상 기본 CRUD API 작성
  • 대담 영상 기본 CRUD API RestDocs 작성
  • 잡페어 인터뷰 기본 CRUD API 작성
  • 잡페어 인터뷰 기본 CRUD API RestDocs 작성

비고

  • 이슈 8번에 명시된 API 개발 및 Restdocs 작성까지 완료되어 PR 올립니다
  • 이후 Video 관련된 API의 경우 새로 배정된 이슈에서 이어서 작업하겠습니다.

2tle added 30 commits July 21, 2024 21:04
[FEAT] JobInterview API, JobInterview RestDocs 작성
@2tle 2tle added 🚀 Feature 기능 개발 🎯 API API 개발 관련 Vido Domain Video 도메인 관련 labels Aug 7, 2024
@2tle 2tle requested a review from yesjuhee August 7, 2024 17:09
@2tle 2tle self-assigned this Aug 7, 2024
@2tle
Copy link
Contributor Author

2tle commented Aug 7, 2024

CI 과정에서 문제가 생길줄을 몰랐네요..
로컬에서 돌렸을 땐(test) 모두 정상 통과되었습니다.

image

@shj1081
Copy link
Contributor

shj1081 commented Aug 7, 2024

test code 부분에

 requestCookies(
                                cookieWithName("refresh-token")
                                        .description("갱신 토큰")
                        ),
 requestHeaders(
                                headerWithName("Authorization")
                                        .description("access token")
                        ),

가 누락된 것 같아용

@yesjuhee
Copy link
Contributor

yesjuhee commented Aug 8, 2024

베이스 브랜치를 PR 올릴 때는 이슈 번호를 close #8 처럼 이슈 번호 앞에 예약어를 달아주면 PR이 머지될 때 연관 이슈가 자동으로 close 되도록 설정할 수 있습니다! CI부분은 저도 한 번 확인해보겠습니당

@yesjuhee
Copy link
Contributor

yesjuhee commented Aug 8, 2024

아 그리고 마지막으로 /src/main/java/com/scg/stop/domain/video 패키지 위치를 /src/main/java/com/scg/stop/video로 변경해주세요!

@yesjuhee yesjuhee merged commit 5097fc0 into develop Aug 9, 2024
1 check passed
@yesjuhee yesjuhee deleted the feat/8-video-api/base branch August 9, 2024 06:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎯 API API 개발 관련 🚀 Feature 기능 개발 Vido Domain Video 도메인 관련
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🚀 [FEAT] Video 관련 API 개발
3 participants