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

API 배포를 위한 merge 입니다. #78

Merged
merged 211 commits into from
Aug 12, 2024
Merged

API 배포를 위한 merge 입니다. #78

merged 211 commits into from
Aug 12, 2024

Conversation

drbug2000
Copy link
Collaborator

📝 요약

API 개발된 것들 배포합니다.

🔖 변경 사항

  • 카카오 OAuth 로그인
  • 채팅방 API
  • 보이스룸 API
  • JPA 연결
  • 게시물 API
  • 스페이스 생성
  • 정산 API
  • 명세서 바탕으로 API 리팩토링
    등등

✅ 리뷰 요구사항

각자 제작한 기능들을 PostMan 사용해서 테스트 해주시면 감사하겠습니다.



📌 PR 진행 시 이러한 점들을 참고해 주세요

* P1 : 꼭 반영해 주세요 (Request Changes) - 이슈가 발생하거나 취약점이 발견되는 케이스 등
* P2 : 반영을 적극적으로 고려해 주시면 좋을 것 같아요 (Comment)
* P3 : 이런 방법도 있을 것 같아요~ 등의 사소한 의견입니다 (Chore)

arkchive and others added 30 commits July 24, 2024 14:38
…ckEnd into Feat/#43/내가-요청한-정산-리스트-api-개발
add LiveKit dependency in build.gradle
add LiveKit Cloud enviroment value in application.yml
hyunn522 and others added 26 commits August 9, 2024 23:20
exception_handler -> exceptionHandler
Copy link
Collaborator

@seongjunnoh seongjunnoh left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Member

@hyunn522 hyunn522 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@arkchive arkchive left a comment

Choose a reason for hiding this comment

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

Good!

@drbug2000 drbug2000 merged commit 49fbed1 into main Aug 12, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants