Conversation
* chore: 미사용 의존성 제거, framer-motion → motion 마이그레이션, 번들 분할 - googleapis, react-colorful, @dnd-kit/modifiers, ts-loader, vite-plugin-pages 삭제 - framer-motion → motion 통일 (4개 파일 import 변경) - vite manualChunks 설정으로 vendor 청크 분리 (radix, motion, i18n, dnd) - vite.config.ts path/\_\_dirname → import.meta.url 방식으로 타입 에러 해결 * chore: 미사용 Radix UI 패키지 및 데드코드 UI 컴포넌트 삭제 - @radix-ui/react-dialog, tabs, scroll-area, avatar, checkbox, progress, separator, toast 삭제 - 미사용 UI 래퍼 15개 파일 및 use-toast 훅 삭제 - manualChunks에서 삭제된 패키지 참조 제거 * perf: 이미지 PNG → WebP 변환 및 리사이즈 (700KB → 23KB) - thung, icon, gmail, calendar, exit PNG를 WebP로 변환 - 과도한 원본 해상도(1024x1024 등)를 적절한 크기로 리사이즈 - import 경로 .png → .webp 변경
- 출석부 테이블(user_progress_a.php) 컬럼 인덱스를 thead에서 동적 감지
- 출석인정 요구시간 컬럼이 없는 5열 테이블에서 weeklyAttendance가
빈 문자열이 되어 출석인데도 결석으로 표시되던 버그 수정
- 일반 강좌는 _a.php에서 시간 데이터도 직접 추출 (mergeTimeData 제거)
- ko/en/ja/zh 다국어 헤더 매칭 지원
- 비트래킹 강좌 데이터를 chrome.storage 대신 sessionStorage에 캐시
- 로컬 스토리지 누적 방지, 탭 닫으면 자동 소멸
- temp course 시스템(registerTempCourse, cleanupExpiredTempCourses) 제거
- OAuth 비대화형 호출(미로그인 상태)에서 불필요한 에러 로그 제거
- 로그 다운로드 시 트래킹 강의의 스토리지 스냅샷 포함
- mock 빌드 시 강의 페이지에 주차별 출석 디버그 배지 표시
- mock 데이터의 weeklyAttendance를 실제 LMS 형식(O/X)으로 수정
- detectColumns 테스트 추가 (5열/6열, 4개 언어, 오매칭 방지)
- parseAttendanceTable 함수 분리 (테스트 가능하도록)
- 6열 테이블: rowspan 전파, 출석인정 요구시간/총 학습시간 추출 검증
- 5열 테이블: 출석인정 요구시간 없는 실제 LMS 구조 기반 테스트
- 미시청("-") 강의, 빈 출석 행 스킵 등 엣지케이스 포함
6-keem
added
bug
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Summary
출석인정 요구시간컬럼이 없는 5열 테이블에서weeklyAttendance가 빈 문자열이 되어 출석인데도 결석으로 표시되던 버그 수정. thead 헤더 텍스트 기반으로 컬럼 인덱스를 동적 감지하여 5열/6열 모두 처리 (ko/en/ja/zh)user_progress_a.php에서 출석 + 시간 데이터를 모두 추출하도록 변경.mergeTimeData제거Test plan