Skip to content

[#147] 시간이 등록되지 않은 강의 표시#169

Merged
halionaz merged 8 commits intodevfrom
#147/show-no-scheduled-lec
Jan 31, 2025
Merged

[#147] 시간이 등록되지 않은 강의 표시#169
halionaz merged 8 commits intodevfrom
#147/show-no-scheduled-lec

Conversation

@halionaz
Copy link
Member

📌 내용

  • 스케줄 되어있지 않은 강의를 시간표 하단에 추가해서 보여줘요
  • Timetable 컴포넌트의 코드양이 너무 방대해지고, 중복되는 코드가 있어 컴포넌트를 분리하는 작업을 진행했어요.
    • 이 PR에 새롭게 작성되었다고 표시되는 코드의 대부분은
      기존에 components/timetable/Grid/LectureSticker.tsx에 있던 코드들이에요
    • 추가적으로 코드 리팩토링을 할만한 포인트를 정리해 다른 이슈로 분리했어요. Timetable Components 정리 #168
Screenshot 2025-01-20 at 5 50 35 PM

☑️ 체크 사항

  • 시간이 등록되지 않은 강의가 잘 표시되는지
  • 이외의 시간표 기능들은 변함없이 정상적으로 작동하는지

❗ Related Issues

@halionaz halionaz requested a review from Virtuso1225 January 20, 2025 08:55
@halionaz halionaz self-assigned this Jan 20, 2025
shadow(),
)}
>
{modalType === 'name' && (
Copy link
Contributor

Choose a reason for hiding this comment

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

p5; return 이전에 타입에 따라서 컴포넌트 분기치는 함수 만들어 두고 사용하면 어떠할까~ 라는

@halionaz halionaz merged commit 8c46cdf into dev Jan 31, 2025
1 check passed
@halionaz halionaz deleted the #147/show-no-scheduled-lec branch January 31, 2025 07:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

일정 미지정인 강의에 대한 표시

2 participants