-
Notifications
You must be signed in to change notification settings - Fork 4
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
V1.2.0 #313
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
방송 파일 경로가 기존 broadcast 이름으로 만들어진 디렉토리에서 내부 추가 디렉토리 경로가 생김에 따라 broadcast properties에 broadcastPath를 추가하였다. Co-authored-by: 최호빈 <[email protected]>
기존 console로 로그를 출력하던 로직에서 logger를 이용해 기록하는 방식으로 변경하였습니다. Co-authored-by: 최호빈 <[email protected]>
CPU 사용량을 줄이기 위해 FFMpeg으로 m4v 파일을 만드는 과정에서 480p 생성 코드를 제거하였습니다. Co-authored-by: 최호빈 <[email protected]>
기존 중복된 코드였던 startStreaming과 endStreaming을 streamingEvent 함수 하나로 변경하였고, 전달받는 인자값을 streamKey와 internalPath를 함께 받을 수 있도록 객체 형태로 전달받도록 변경하였습니다. Co-authored-by: 최호빈 <[email protected]>
this.storagePath가 정해지기 전 종료 관련 이벤트가 발생했을 때 storagePath가 undefined으로 처리되는 현상을 막기 위한 소켓 이벤트 분리와 방송 종료 시 방송 디렉토리를 제거하던 로직을 삭제하고, 방송 별 디렉토리 분리로 변경함에 따라 internalPath를 방송 시작 시 함께 전달하도록 변경하였습니다. Co-authored-by: 최호빈 <[email protected]>
- 복사 버튼 컴포넌트 작성 - 스트림 url, 키 api 추후 붙여야함
- img에 svg 덧대야함
- 이미지 remotePattern에 avatars.githubusercontent.com을 추가
- 채팅창 채팅 목록 UI 중 '님' 제거 #279
- stroke-width -> strokeWidth - px -> rem으로 단위 변경
- mydata 타입 추가 - msw, actions에 핸들러 추가 - 재발급 버튼 추가(로직 미구현) - 재발급 버튼 누를시 나올 모달도 구현해야함
- 콘텐츠 카테고리 상수 예시 생성 #263
`/live/category?content=music&mood=happy`와 같이 카테고리를 포함한 요청이 왔을 때 해당 카테고리에 맞는 방송 목록을 전달할 수 있도록 Controller와 Service 로직을 작성하였습니다.
카테고리 방송 목록이 전달된 양식에 맞게 반환되는지 확인하는 테스트를 작성하였습니다.
live.controller의 category 라우트에 / 문자를 추가하였습니다.
stream key를 갱신하고 반환하는 api를 구현하였습니다.
채팅 서버에서 반환하는 값에 color값을 추가하였고, color를 랜덤으로 반환하는 함수를 구현하였습니다.
- 기존의 DataSource 주입 방식은 문자열 토큰 방식이었다. 이 방식은 여러 DataSource 즉, 여러 데이터베이스를 주입하려할 때 각 DataSource를 구분하는 용도로 주로 사용하므로 하나의 데이터베이스만 사용하고 있는 우리 서비스에서는 DataSource라는 클래스를 참조하는 것이 더 적합하다고 생각하여 변경하였다.
- follow에 해당하는 path alias를 추가하였다.
- 로그인을 한 유저는 멤버를 팔로우/언팔로우 할 수 있다. - 만약 팔로우를 이미 한 상태에서 팔로우 요청을 하면 BAD_REQUEST 오류를 발생시킨다. - 만약 팔로우를 하지 않은 상태에서 언팔로우 요청을 하면 BAD_REQUEST 오류를 발생시킨다.
- 본인을 팔로우/언팔로우 하지 못하도록 막았다.
- 사용자의 팔로워와 팔로잉 목록을 가져온다.
naver 로그인 구현에 따라 package.json과 tsconfig.json에 naver path alias를 추가하였습니다.
github 로그인 service 로직에 axios시 2xx 응답이 돌아오지 않아 프로세스가 종료되는 것을 방지하기 위해 catch로 에러를 처리할 수 있는 로직을 추가했습니다. 추가로 getUserInfo의 authorization header의 token type을 Token -> Bearer로 변경하였습니다.
제목이 곧 내용입니다.
pm2 reload -> pm2 reload ecosystem.config.js 로 변경했습니다.
- 응답으로 OK 객체를 반환한다.
본문을 포함하기 위해 204 -> 200으로 변경하였습니다.
방송 중일 때 Not Found를 반환하던 문제를 해결했습니다.
정적으로 생성해서 전달하는 이미지 URL의 Path를 변경하였습니다.
이미지를 전달받았을 때 확장자를 파악하고 이미지 데이터만 저장하도록 변경하였습니다.
이미지 확장자 반환 코드 변경
WilleLee
approved these changes
Nov 28, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
다들 고생하셨습니다~ LGTM FUNCH! 👊🏻
zzawang
approved these changes
Nov 28, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
6시간 동안 수고하셨습니다!! LGTM 👍🏻👍🏻👍🏻
다들 고생이 많아요ㅠ 그래도 완성되가는거보니까 뿌듯해요 좀만 더 화이팅! |
HongBoogie
approved these changes
Nov 28, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.