Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feat] 스케쥴러 반응형 슬롯 #684 #690

Conversation

42sungwook
Copy link
Contributor

@42sungwook 42sungwook commented Feb 27, 2023

📌 개요

스케쥴러 반응형 슬롯 제작

💻 작업사항

  • current 슬롯/ preview 슬롯 컴포넌트 나눠서 관리
    • current 슬롯 - api로 불러온 현재 만들어져있는 슬롯
    • preview 슬롯 - 임의로 미리보기 하기 위해 만드는 슬롯
  • 현재 시간은 파란색/볼드체로 표기
  • n 시간 뒤의 input 추가
  • scheduler input 정보로 슬롯이 반응형으로 바뀌게 설정

실행 방법

  1. npm i && npm run dev
  2. 좌측 사이드바에서 스케쥴러 관리나 URL에 http://localhost:3000/admin/scheduler 입력

ETC

  • 아직 작업할게 좀 남아있습니다 (noSlot 안보여주기, 날짜 보이기, 게임시간 바꾸기 등등)
  • 고칠 점도 꽤 많다고 생각합니다 (ex. 중복되는 덧셈 변수로 처리하기, 타입들 따로 파일로 빼서 관리하기)
  • 부족한 점 인지하고 있어 고칠 필요가 있어 보이는 부분은 팍팍 지적해주시면 감사하겠습니다😄

@42sungwook 42sungwook requested a review from a team as a code owner February 27, 2023 07:34
@42sungwook 42sungwook requested review from mike2ox, raehy19 and Wilbur0306 and removed request for a team February 27, 2023 07:34
@42sungwook 42sungwook changed the title [Feat] 스케쥴러 반응형 슬롯 [Feat] 스케쥴러 반응형 슬롯 #663 Feb 27, 2023
@42sungwook 42sungwook self-assigned this Feb 27, 2023
@42sungwook 42sungwook added enhancement New feature or request admin 관리자 페이지에서 발생한 issue labels Feb 27, 2023
@42sungwook 42sungwook changed the title [Feat] 스케쥴러 반응형 슬롯 #663 [Feat] 스케쥴러 반응형 슬롯 #684 Feb 27, 2023
Copy link
Contributor

@raehy19 raehy19 left a comment

Choose a reason for hiding this comment

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

브랜치 옮겨서 작업 이어서하시는거죠 ? 파이팅입니다 !!

@42sungwook
Copy link
Contributor Author

브랜치 옮겨서 작업 이어서하시는거죠 ? 파이팅입니다 !!

넵 브랜치 새로 만들어서 작업하려고요! 감사합니당

Copy link
Contributor

@Wilbur0306 Wilbur0306 left a comment

Choose a reason for hiding this comment

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

오늘 전체 회의 때 나눈 얘기들 보면 로직이 결코 쉬워보이지 않는데, 코드를 보니 벌써 큰 부분들을 많이 자리 잡아놓으신 것 같습니다! PR 올린 설명에서 언급한 버그들 수정하는거 파이팅입니다!!

@42sungwook 42sungwook merged commit 49a01b9 into Feat/스케쥴러-페이지-제작-#663 Feb 28, 2023
@42sungwook 42sungwook deleted the 684-feat-스케쥴러-새로-생성되는-슬롯 branch March 7, 2023 06:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
admin 관리자 페이지에서 발생한 issue enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants