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

api 구현 완료 및 백엔드 배포 준비 #167

Merged
merged 19 commits into from
Nov 14, 2024
Merged

api 구현 완료 및 백엔드 배포 준비 #167

merged 19 commits into from
Nov 14, 2024

Conversation

happyhyep
Copy link
Member

📝 PR 개요

  • api 구현
  • swagger 문서 작성
  • postman으로 api 테스트

🔍 변경 사항

  • 사용자 로그인 api 구현

  • 새로운 채널 생성 api 구현

  • 게스트 추가 api 구현

  • 채널 정보 조회 api 구현

  • 게스트 정보 조회 api 구현

  • 사용자가 host인 채널 목록 반환 api 구현

  • swagger 문서 작성

  • postman으로 api 테스트

✅ 체크리스트 (Checklist)

  • 코드가 빌드 오류 없이 잘 작동하는지 확인
  • 테스트가 통과하는지 확인
  • 스타일 가이드와 일관성을 유지했는지 확인
  • 관련 문서가 업데이트되었는지 확인 (선택 사항)
  • 리뷰어가 이해할 수 있도록 주석이나 설명을 추가했는지 확인

🔄 관련 이슈 (Linked Issues)

#166

happyhyep and others added 19 commits November 12, 2024 13:37
@happyhyep
[BE][Feat] : #42 : 로그인 api swagger 문서 작성
6070190 
- 로그인 api swagger 문서 작성
@happyhyep
Merge pull request #147 from boostcampwm-2024/feature/be/#42-loginapi
ecd0e82 
[BE][Feat] : #42 : 로그인 api swagger 문서 작성
@happyhyep
[BE][Feat] : #49 : 경로방 생성 api
a44fcaf 
- uuid 생성을 위한 패키지 설치
- eslint 백엔드 설정 수정
- 채널 생성 api 구현 (router, controller, service, repository)
- guest 추가 api 구현
- postman 테스트
@happyhyep
Merge pull request #148 from boostcampwm-2024/feature/be/#49-#50-add-…
815edd7 
…channel

[BE][Feat] : #49 : 경로방 생성 api
@happyhyep
[BE][Docs] : swagger 문서 작성 및 수정
877dbfc 
- login api에 대한 swagger 문서 수정
- channel 생성 api에 대한 swagger 문서 생성
@happyhyep
[BE][Docs] : swagger 문서 작성 및 수정
ba8c47f 
- login api에 대한 swagger 문서 수정
- channel 생성 api에 대한 swagger 문서 생성
@happyhyep
Merge pull request #149 from boostcampwm-2024/feature/be/#49-#50-add-…
21a29ee 
…channel

[BE][Feat] : swagger 문서 작성 및 수정
@happyhyep
[BE][Feat] : 채널에 guest 추가 api 구현
b197081 
- 채널에 guest 추가하는 api에 대한 swagger 문서 추가
- guest 추가하는 api 구현
@happyhyep
Merge pull request #151 from boostcampwm-2024/feature/be/#49-#50-add-…
f1ef1c9 
…channel

[BE][Feat] #50 : 경로방 별 사용자 추가 post api
@happyhyep
[BE][Feat] #45 : 채널 id로 채널 정보, guest 정보 get API 구현
4d20551 
- 채널 id로 채널 정보, guest 정보 get API에 대한 swagger 문서 추가
- 채널 id로 채널 정보, guest 정보 get API 구현
@happyhyep
Merge pull request #152 from boostcampwm-2024/feature/be/#45-getChann…
5f37574 
…elInfo

[BE][Feat] #45 : 채널 id로 채널 정보, guest 정보 get API 구현
@happyhyep
[BE][Feat] #46 : guest id로 guest 정보 get API 구현
5d1ec49 
- guest id로 guest 정보 get API에 대한 swagger 문서 추가
- guest id로 guest 정보 get API 구현
- swagger 문서 세팅 수정
@happyhyep
Merge pull request #156 from boostcampwm-2024/feature/be/#46-getGuest…
496fb13 
…Data

[BE][Feat] #46 : guest id로 guest 정보 get API 구현
@happyhyep
[BE][Feat] #43 : user id로 해당 user가 생성한 channel 정보 가져오기 API 구현
006842c 
- user id로 해당 user가 생성한 channel 정보 가져오기 API에 대한 swagger 문서 추가
- user id로 해당 user가 생성한 channel 정보 가져오기 API 구현
- API 형식 통일
@happyhyep
[BE][Feat] #43 : user id로 해당 user가 생성한 channel 정보 가져오기 API 구현
e4b016a 
- user id로 해당 user가 생성한 channel 정보 가져오기 API에 대한 swagger 문서 추가
- user id로 해당 user가 생성한 channel 정보 가져오기 API 구현
- API 형식 통일
@happyhyep
Merge pull request #157 from boostcampwm-2024/feature/be/#43-getUserC…
c2ab9cd 
…hannel

[BE][Feat] #43 : user id로 해당 user가 생성한 channel 정보 가져오기 API 구현
@happyhyep
[BE][Feat] #159 : 회원가입 api 구현
06e3e19 
- 회원가입 api에 대한 swagger 문서 추가
- 회원가입 api 구현
- auth에 대한 jsdoc 추가
- 회원가입시 비밀번호 암호화해서 db에 저장
@happyhyep
[BE][Refactor] swagger 문서 tag 추가, jsdoc 추가
1d6c50a 
- channel api에 대한 swagger 문서 tag 추가
- controller, service, repository에 대한 jsdoc 추가
- index.js 불필요한 코드 제거
@happyhyep
Merge pull request #160 from boostcampwm-2024/feature/be/#158-registe…
d09b862 
…rUser

[BE][Feat] #159 : 회원가입 api 구현, [BE][Refactor] swagger 문서 tag 추가, jsdoc 추가
@happyhyep happyhyep added the 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전) label Nov 14, 2024
@happyhyep happyhyep self-assigned this Nov 14, 2024
effozen
effozen approved these changes Nov 14, 2024
View reviewed changes
Copy link
Collaborator

@effozen effozen left a comment

Choose a reason for hiding this comment

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

백앤드 전체 코드를 Dev 브렌치에 통합하는 과정 확인했습니다.

프론트앤드 코드에서 설정 등이 변한게 있어서 충돌될까봐 두렵긴 한데..
사실 백이 설정 쪽에서 거의 원본 코드에 가까워서 백을 먼저 머지하고 여기에 맞추는게 맞지 않을까해서

백앤드 먼저 머지 -> 프론트 이후 머지 이렇게 가야할 듯 싶습니다.

@happyhyep
Copy link
Member Author

백앤드 전체 코드를 Dev 브렌치에 통합하는 과정 확인했습니다.

프론트앤드 코드에서 설정 등이 변한게 있어서 충돌될까봐 두렵긴 한데.. 사실 백이 설정 쪽에서 거의 원본 코드에 가까워서 백을 먼저 머지하고 여기에 맞추는게 맞지 않을까해서

백앤드 먼저 머지 -> 프론트 이후 머지 이렇게 가야할 듯 싶습니다.

넵 저도 이렇게 생각해서, 빠르게 backend 먼저 dev로 머지하고 front 병합하는게 좋을 것 같습니다. 다른 분들 approve 해주시면 바로 머지하겠습니다!

leedongyull
leedongyull approved these changes Nov 14, 2024
View reviewed changes
Copy link
Collaborator

@leedongyull leedongyull left a comment

Choose a reason for hiding this comment

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

LGTM

@juwon5272
Copy link
Collaborator

juwon5272 commented Nov 14, 2024 •
edited
Loading

지금 동율님 PR에서 BE쪽 package.json을 다시 작성하신게 있어서 지금 합치면 충돌나겠네요. BE는 dev 브랜치에 넣어두고 FE는 그거 고쳐지는대로 같이 합쳐야할거 같습니다. 고생하셨어요!

@happyhyep happyhyep merged commit 6907ab5 into development Nov 14, 2024
5 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@effozen effozen effozen approved these changes

@juwon5272 juwon5272 juwon5272 approved these changes

@leedongyull leedongyull leedongyull approved these changes

Assignees

@happyhyep happyhyep

Labels
확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@happyhyep @juwon5272 @leedongyull @effozen