-
Notifications
You must be signed in to change notification settings - Fork 2
[4팀] 프론트엔드 코드리뷰용 PR #35
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
base: review
Are you sure you want to change the base?
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
[SRLT-83] 요금제 페이지 width 삭제
[SRLT-77] 마이페이지 사업계획서 목록 연동
…nto SRLT-84-전문가-피드백-요청
[SRLT-88] 기타 에러 수정
…nto SRLT-84-전문가-피드백-요청
[SRLT-84] 전문가 피드백 요청
[SRLT-104] 수정사항 반영 및 update README, 메타태그 생성
[SRLT-107] 이미지 캡션 추가
[SRLT-104] 맞춤형피드백 이미지 변경
[SRLT-107] 에디터 및 pdf 오류 수정
[SRLT-107] placeholder 변경 및 코드 구조 개선
[SRLT-107] 마이페이지 pdf 구분 처리
[SRLT-107] 보기 상태 초기화
[SRLT-109] 전문가 상세페이지 UI
[4팀] 스타라이트 프론트엔드
✨ 리뷰를 요청드리는 주요 부분
가장 고민되는 지점을 아래 형식으로 정리했습니다.
src/lib/imageUpload.ts: URL 인코딩/디코딩 로직이 복잡한데, 더 명확한 방식이 있는지 궁금합니다.src/lib/pdfDownload.ts: 에디터에서 작성한 사업계획서를 PDF로 추출할 때 제목 영역의 스타일과 표 스타일이 깨지는 문제가 발생하고 있습니다. 또한, 업로드한 이미지가 PDF 내에서 흰색으로 표시되는 현상(외부 도메인에 업로드된 이미지의 CORS 정책 이슈로 예상)이 있습니다. 이 두 가지 문제를 어떻게 해결할 수 있을지 문의드립니다. 추가로, 생성되는 PDF의 용량을 줄이는 방법에 대해서도 알고 싶습니다.src/lib/business/requestBuilder.ts&src/lib/business/editorContentMapper.ts: TipTap JSON을 API 요청 형식으로 변환하는 로직이 복잡한데, 분리 및 개선의 여지가 있는 지 궁금합니다.💬 4팀이 고민하고 있는 부분
에러 처리: 현재 많은 곳에서
console.error와alert를 사용하는데, 통일된 에러 처리 전략(에러 바운더리, 토스트 등)을 도입해야 하는지lib/business폴더 구조 및 모듈 분리: 사업계획서 작성 관련 모든 함수들을lib/business폴더에 평면적으로 배치했습니다. 현재 파일 구성은 다음과 같습니다:extensions.ts(TipTap 커스텀 확장),useEditorConfig.ts(에디터 설정)editorContentMapper.ts(JSON → API 형식),editorToHtml.ts(JSON → HTML)requestBuilder.ts(요청 바디 생성),postSpellCheck.ts(맞춤법 검사 페이로드)getChecklist.ts(체크리스트 추출),getSubsection.ts(번호 → 타입 매핑)기능별로 하위 폴더로 분리할지(예:
editor/,converter/,api/,utils/), 현재처럼 평면 구조를 유지할지, 또는 다른 분리 기준이 있는지 조언 부탁드립니다. 또한 각 파일의 책임 범위가 적절한지, 일부 함수들을 더 세분화하거나 통합할 여지가 있는지도 궁금합니다.💌 요청드리는 리뷰 방향
바쁘신 와중에 시간 내어 리뷰해주셔서 감사합니다.
편하게 피드백 부탁드립니다! 🥹