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.
✨ 해당 이슈 번호 ✨
#129
todo
📌 내가 알게 된 부분
📌 공유하고 싶은 부분
기존에 회의 입장할 때 방장이 입력한 회의명이 아니라 '에이셉 전체회의' 라는 문구가 하드코딩 되어있던 것이 문제였는데요.
회의 입장페이지에서 해당 회의가 유효한 회의인지 (이미 확정된 회의인지) 확인하는 api를 쏘는데, 해당 요청에 대한 응답으로 회의명을 받아올 수 있도록 서버측에 요청해서 해결했습니다.
찬우오빠가 모든 퍼널의 공통적인 요소들을 SteppingStone이라는 공통컴포넌트로 구현했는데, 그래서 회의명에 해당하는 부분도 'SteppingBody'라는 공통 컴포넌트로 나타내고 있었습니다. 이 컴포넌트에 api 응답으로 받아온 회의명을 prop으로 넘겨주어야해서, SteppingBody 컴포넌트에 meetingTitle이라는 prop을 추가했고, 회의 입장 페이지 외에는 이 prop이 필요하지 않기 때문에 type은 undefined를 허용했습니다.
워낙 공통컴포넌트가 깔쌈하게 되어있었어서 이 한 페이지를 위해 prop을 추가한게 썩 만족스럽지는 않은데... 전역상태관리로 해결했어야하나 싶기도 하구요...
📌 질문할 부분
📌스크린샷
2023-09-13.5.39.50.mov
입장페이지 새로고침하는 영상인데, 보다시피 api 응답 받아오는데 시간이 걸려서 잠깐동안 빈 글자로 나옵니다 ㅠㅠ
흠 이걸 어떻게 해결하면 좋을지....