404 페이지, 존재하지 않는 스페이스 페이지 핸들링 #166
Merged
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
hoqn
requested review from
parkblo and
CatyJazzy
and removed request for
a team
parkblo
approved these changes
Nov 28, 2024
Comment on lines
+21
to
+26
| {RestoreActions && ( | ||
| <div className="mt-12"> | ||
| <RestoreActions /> | ||
| </div> | ||
| )} | ||
| </div> |
There was a problem hiding this comment.
RestoreAction을 따로 받아서 더욱 재사용성이 높아진 것 같습니다 !
Comment on lines
+41
to
+47
| provider.once("connection-close", (event: CloseEvent) => { | ||
| if (event.code === 1008) { | ||
| provider.shouldConnect = false; | ||
| setError(new Error("찾을 수 없거나 접근할 수 없는 스페이스예요.")); | ||
| } | ||
| }); | ||
|
|
There was a problem hiding this comment.
(0.3점) 1008 코드 이외의 코드가 발생할 시의 대응을 추가해볼 수도 있을 것 같습니다!
There was a problem hiding this comment.
넵! 현재 백엔드에서 존재하지 않는 스페이스는 1008 코드 주면서 연결을 끊고 있는데, 다른 경우엔 "알 수 없는 오류"처럼 처리해야 할 것 같아요.
대신 이렇게 shouldConnect = false를 해줘도 될지 모르겠네요. 기본적으로 계속 연결 재시도를 하더라고요. 1008번의 경우 확실히 재연결이 필요가 없어서 이렇게 했습니다
헤드 노드 스켈레톤을 아예 뺄까 생각 중입니다. 로딩바나 안내 문구 넣기엔 너무 짧은 시간이라 스켈레톤이 낫지 않을까 했는데 그러기조차도 너무 짧은 시간이네요.
대신 이렇게 로더를 헤드노드 크기에 맞춰서 해놓는 건 어떨까 고민 중입니다. (이게 아예 회전하는 건데 gif로 하다보니까 뭔가 프레임이 이상하네요) |
CatyJazzy
approved these changes
Nov 28, 2024
hoqn
force-pushed
the
feature/kindness
branch
from
6e1d0fc to
00131d1
Compare
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.
✏️ 한 줄 설명
데모 공개에 앞서, 자연스러운 사용자 경험을 위해 에러 페이지와 로딩 상태를 구현하고자 했어요.
✅ 작업 내용
🏷️ 관련 이슈
📸 스크린샷/영상
404 페이지와 존재하지 않는 스페이스 접속 시에 이렇게 뜹니다.
로딩할 떄 하얀 화면이 뜨는 게 좀 그래서.. 로딩 바를 넣을까 하다가 그냥 스켈레톤처럼 처리해봤습니다
📌 리뷰 진행 시 참고 사항
이건 같이 기여해서 수정해도 좋을 것 같아요 -> 데모 공개 전에 조금 매끄럽지 않은 부분..