Refector : 강의코드 강의자,참여자 페이지에 반영, 화이트보드 공유 과정 안정화 #227
Merged
Conversation
This file contains hidden or 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
강의 페이지의 쿼리스트링에 맞춰서 강의코드를 표시하고, 복사 내용에도 반영합니다.
play.svg 아이콘이 진상님 PR에서 덮어씌워져서 생긴 충돌을 복구했습니다.프로그래스바에서 사용하는 play아이콘에는 progress를 붙였습니다.
…류 해결 setLectureCode위치를 수정해서 오류를 해결했습니다.
볼륨바에 내용을 전달하는 과정을 recoil을 이용하여 단순화했습니다.
…231207-enter-room
서버에 참여자가 처음 입장 했을 때, 강의자가 마지막으로 보낸 화이트보드 데이터를 받아서 화면에 그릴 수 있습니다.
…231207-enter-room
비디오 스트림 형태로 전송/수신하는 과정을 제거했습니다.
동령님이 수정하신 코드를 이용하여 toast를 같은 함수에서 여러 개 올릴 수 있게 했습니다. Co-authored-by: LellowMellow <[email protected]>
사용하지 않는 주석과 콘솔을 제거했습니다.
✅ Deploy Preview for boarlog ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
…231207-enter-room
Jw705
requested review from
Byeonjin and
LellowMellow
and removed request for
Byeonjin
Byeonjin
reviewed
Dec 7, 2023
| @@ -1,7 +1,7 @@ | |||
| import { useState, useEffect } from "react"; | |||
| import { useSetRecoilState } from "recoil"; | |||
| import selectedMicrophoneState from "../../../stores/stateSelectedMicrophone"; | |||
| import micVolmeState from "../../../stores/stateMicVolme"; | |||
| import micVolmeGainState from "../../../stores/stateMicVolumeGain"; | |||
Byeonjin
reviewed
Dec 7, 2023
| height: 0 | ||
| }; | ||
| //{ newData }: { newData: any } | ||
| const randerCanvas = (newData: ICanvasData) => { |
There was a problem hiding this comment.
rander -> render 이렇게 바꿔주시면 좋을 것 같습니다!
제정신이 아니였나 봅니다. 오타가 무려 2개나..
LellowMellow
approved these changes
Dec 7, 2023
| @@ -15,6 +15,7 @@ interface HeaderProps { | |||
| const Header = ({ type }: HeaderProps) => { | |||
| const [isProfileClicked, setIsProfileClicked] = useState(false); | |||
| const [isSettingClicked, setIsSettingClicked] = useState(false); | |||
| const [lectureCode, setLectureCode] = useState("000000"); | |||
There was a problem hiding this comment.
흑흑... API 빨리 연결해서 해결하겠습니다 :)
| @@ -11,7 +11,7 @@ const HeaderCodeCopyButton = ({ lectureCode }: HeaderCodeCopyButtonProps) => { | |||
| const handleShareButtonClicked = async () => { | |||
| try { | |||
| await navigator.clipboard.writeText( | |||
| `[함께 듣는 실시간 강의 Boarlog]\n\n지금 진행되는 강의에 참여해 보세요.\n\n강의 코드: ${lectureCode}\n강의 제목: 화이트보드 테스트\n강의 링크: https://boarlog.netlify.app/participant` | |||
| `[함께 듣는 실시간 강의 Boarlog]\n\n지금 진행되는 강의에 참여해 보세요.\n\n강의 코드: ${lectureCode}\n강의 제목: 화이트보드 테스트\n강의 링크: https://boarlog.netlify.app/participant?roomid=${lectureCode}` | |||
There was a problem hiding this comment.
이 부분까지 미리 반영해주셨군요!! 감사합니다 :)
Byeonjin
reviewed
Dec 7, 2023
| @@ -9,12 +9,15 @@ interface UseToastProps { | |||
| } | |||
|
|
|||
| export const useToast = () => { | |||
| const [toastList, setToastList] = useRecoilState(toastListState); | |||
| const [_, setToastList] = useRecoilState(toastListState); | |||
There was a problem hiding this comment.
value를 사용하지 않으시기 때문에 _로 바꾸신거라면 useRecoilState 함수 대신 useSetRecoilState를 사용하시는 건 어떨까요?
Byeonjin
reviewed
Dec 7, 2023
Comment on lines
+1
to
+2
| import PlayIcon from "@/assets/svgs/progressPlay.svg?react"; | ||
| import PauseIcon from "@/assets/svgs/progressPause.svg?react"; |
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.
작업 개요
작업 사항
여기에 작성하세요
고민한 점들(필수 X)
여기에 작성하세요
스크린샷(필수 X)
231207-2.mp4