Skip to content

Conversation

@yiseungyun
Copy link
Member

@yiseungyun yiseungyun commented Dec 3, 2024

Note

  • 작성자: 이승윤
  • 작성 날짜: 2024.12.03

✅ 체크리스트

  • 코드가 정상적으로 작동하는지 확인했습니다.
  • 주요 변경사항에 대한 설명을 작성했습니다.
  • 코드 스타일 가이드에 따라 코드를 작성했습니다.

🧩 작업 내용

  • inProgress 상태에 따라 요청
  • NotFound 컴포넌트 줄바꿈 지원

📝 작업 상세 내역

inProgress 상태에 따라 요청

#315
세부 내용은 여기에 있습니다. 위 상태에서 탭에 실제 inProgress가 반영된 결과가 나옵니다.

@백엔드 추후 해당 부분 페이지네이션이 필수적일거 같습니다. 지금은 규모가 작아서 세션 생성 수가 작겠지만, 세션이 많을 때 한번에 받아오면 안될거 같아요! 일단 나중에 구현하는거로 하겠습니다.

NotFound 컴포넌트 줄바꿈 지원

스크린샷 2024-12-03 오후 7 53 38 스크린샷 2024-12-03 오후 8 04 44

정우님이 만드신 notFound 컴포넌트가 중앙 정렬이 안되길래 보니까 리스트 페이지 자체에 그리드가 되어있어서 리스트가 없을 때는 그리드 안쓰도록 해서 중앙 정렬했습니다. 또 텍스트 자체 줄바꿈이 되면 깔끔할 거 같아 줄바꿈되도록 수정했습니다! + 중앙 정렬하고 돋보기 이미지와 비슷한 색상으로 맞췄습니다.

추가적으로 원래 새로운 세션 생성하기 버튼이 존재했는데 바로 위에 떠서 중복될거 같아 이 부분에 버튼은 없앴습니다.

💬 다음 작업 또는 논의 사항

  • 시간이 될지 모르겠는데 join room할 때 닉네임 입력하는 부분을 없애는 방향으로 수정해야할 생각입니다..

@yiseungyun
Copy link
Member Author

충돌 해결하고 오겠심

@ShipFriend0516
Copy link
Member

검색 기능도.. 언젠가!!

Copy link
Member

@ShipFriend0516 ShipFriend0516 left a comment

Choose a reason for hiding this comment

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

👍

Copy link
Collaborator

@blu3fishez blu3fishez left a comment

Choose a reason for hiding this comment

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

image

현재 확인해보니 0 과 1로 보내지고 있는 것 같은데여 이전 PR에도 언급드렸다시피 true false로 하면 좋을 듯 합니다.

아니면 0 과 1이 좋을까요?

@yiseungyun
Copy link
Member Author

@blu3piece 헉 다 false/true로 했는데 확인해보겠습니당

@blu3fishez
Copy link
Collaborator

@blu3piece 헉 다 false/true로 했는데 확인해보겠습니당

제 착오였읍니다.... 머지할게욤

@blu3fishez blu3fishez merged commit 76a6033 into dev Dec 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants