Skip to content

Refactor: 강의자/참여자 헤더 리팩토링 #233

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
Merged
merged 6 commits into from
Dec 9, 2023
Merged

Refactor: 강의자/참여자 헤더 리팩토링 #233

merged 6 commits into from
Dec 9, 2023

Conversation

Jw705
Copy link
Collaborator

@Jw705 Jw705 commented Dec 8, 2023 •
edited
Loading

작업 개요

강의자와 참여자 헤더의 코드를 간단하게 리팩토링 했습니다.
다음 PR 올릴 때 변경 사항이 많으면 코드리뷰 하시기 힘드실 것 같아서 리팩토링 한 내용만 먼저 PR 올립니다.
기능이 변한 것은 없어서 편하게 LGTM만 해주셔도 감사하겠습니다.

작업 사항

  • socket 통신 관련 함수를 핸들러 함수로 분리하여 사용하고, 이벤트를 한 곳에서 등록하도록 개선했습니다.
  • 다시보기 페이지에서의 사용을 고려해서 캔버스 데이터를 저장하고 로드하는 함수를 별도의 파일로 분리했습니다.
  • AnalyserNode에서 정규화된 볼륨을 계산하는 calcNormalizedVolume, 헤더 상단 지난 시간을 표시하는 convertMsTohhmm 함수를 만들어 분리했습니다.
  • socketRef2 -> lectureSocketRef로 이름을 변경했습니다.

고민한 점들(필수 X)

여기에 작성하세요

스크린샷(필수 X)

여기에 작성하세요

Jw705 added 5 commits December 8, 2023 21:04
@Jw705
refactor: socket 통신 관련 함수 핸들러 분리
5ec0eb5 
socket 통신 관련 함수를 핸들러 함수로 분리했습니다.
@Jw705
refactor: ICanvasData, saveCanvasData, loadCanvasData 분리해서 fabricCanv…
b2c1ace 
…asUtil에 저장

다시보기 페이지를 구현할 때 함께 사용할 수 있는 함수를 분리해서 같이
사용할 수 있게 했습니다.
@Jw705
refactor: calcNormalizedVolume 분리
4eaea99 
AnalyserNode에서 정규화된 볼륨을 계산하는 calcNormalizedVolume를
분리했습니다.
@Jw705
refactor: 기타 함수 분리 및 정리
275ee0a 
convertMsTohhmm와 같은 함수를 만들어 분리했습니다.
@Jw705
refactor: socketRef2 -> lectureSocketRef로 이름 변경
7556df9 
두번째로 연결해서 사용하는 소켓의 이름을 더 명확한 의미를 갖도록
변경했습니다.
@Jw705 Jw705 added 🛠️ Refactor 리팩토링 작업 FE 프론트엔드 작업 labels Dec 8, 2023
@Jw705 Jw705 added this to the 5주차 milestone Dec 8, 2023
@Jw705 Jw705 self-assigned this Dec 8, 2023
@netlify Netlify
Copy link

netlify bot commented Dec 8, 2023 •
edited
Loading

✅ Deploy Preview for boarlog ready!

Name Link
🔨 Latest commit 635e1c5
🔍 Latest deploy log https://app.netlify.com/sites/boarlog/deploys/6573197a336a6b000813b28a
😎 Deploy Preview https://deploy-preview-233--boarlog.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@Jw705
fix: 리팩토링 이후 변경되지 않은 캔버스 내용도 전송하는 오류 해결
635e1c5 
반환값을 주는 것을 잊고 있다가 발견해서 수정했습니다.
Byeonjin
Byeonjin approved these changes Dec 9, 2023
View reviewed changes
Copy link
Collaborator

@Byeonjin Byeonjin left a comment

Choose a reason for hiding this comment

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

lgtm!

LellowMellow
LellowMellow approved these changes Dec 9, 2023
View reviewed changes
Copy link
Collaborator

@LellowMellow LellowMellow left a comment

Choose a reason for hiding this comment

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

파일 분리와 코드 method 분리 너무 좋습니다! 너무 늦게 코드리뷰 드려서 죄송합니다! LGTM :)

@Jw705 Jw705 merged commit 00b2d50 into boostcampwm2023:dev Dec 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Byeonjin Byeonjin Byeonjin approved these changes

@LellowMellow LellowMellow LellowMellow approved these changes

Assignees

@Jw705 Jw705

Labels
FE 프론트엔드 작업 🛠️ Refactor 리팩토링 작업
Projects
None yet
Milestone
5주차
Development

Successfully merging this pull request may close these issues.
3 participants
@Jw705 @Byeonjin @LellowMellow