Skip to content

Conversation

@yiseungyun
Copy link
Member

@yiseungyun yiseungyun commented Dec 3, 2024

관련 이슈 번호

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

✅ 체크리스트

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

🧩 작업 내용

  • 질문지 생성 페이지 리팩토링
  • 세션 생성 페이지 리팩토링
  • 세션 관련 코드 세션 페이지 폴더로 이동

📝 작업 상세 내역

질문지 생성 페이지 리팩토링

  • 비즈니스 로직이랑 분리하려했는데, 여기 딱히 없어서 view에는 질문지 생성 페이지에 사용하는 컴포넌트 넣어뒀습니다.

세션 생성 페이지 리팩토링

  • 질문지 생성 페이지와 동일한 작업을 했습니다.

세션 관련 코드 세션 페이지 폴더로 이동

  • test 코드랑 세션 페이지에서만 쓰는 타입 관련 코드를 세션 페이지 폴더로 이동했습니다.

index.html 파일 오류 수정했습니다. 이제 로고가 뜰거에요. 🐽

💬 다음 작업 또는 논의 사항

  • 스터디 채널 디자인 변경
  • 말할 때 표시하는 부분 조금 수정 예정

@yiseungyun yiseungyun added the 🎁 Refactoring 리팩토링 label Dec 3, 2024
@yiseungyun yiseungyun self-assigned this Dec 3, 2024
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.

리팩토링 수고하셨습니다. 폴더구조 변경하고 한번씩 pnpm build 해보고 빌드오류 없는지 확인도 해보시면 좋아요

@@ -1,43 +1,31 @@
<!doctype html>
<html lang="ko">
</html>
Copy link
Member

Choose a reason for hiding this comment

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

아 이거구나 ㅋㅋㅋㅋ

Copy link
Member Author

Choose a reason for hiding this comment

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

ㅋㅋ 넴 요즘 빌드.... 계속 하고 있습니다 저번주의 기억 때문이랄까....

Copy link
Collaborator

Choose a reason for hiding this comment

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

눈물이 나로심..............

Copy link
Member

Choose a reason for hiding this comment

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

ㅋㅋㅋㅋㅋ 이게 폴더구조 변경하면 가끔 안될때있더라구요 저도 당함 ㅋㅋ

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

Labels

🎁 Refactoring 리팩토링

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants