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

[API] /api/share/timetable/image 구현 #91

Closed
pbc1017 opened this issue Mar 13, 2024 · 3 comments · Fixed by #92
Closed

[API] /api/share/timetable/image 구현 #91

pbc1017 opened this issue Mar 13, 2024 · 3 comments · Fixed by #92

Comments

@pbc1017
Copy link
Contributor

pbc1017 commented Mar 13, 2024

No description provided.

@pbc1017 pbc1017 linked a pull request Mar 13, 2024 that will close this issue
@pbc1017
Copy link
Contributor Author

pbc1017 commented Mar 13, 2024

image (6)

@pbc1017
Copy link
Contributor Author

pbc1017 commented Mar 13, 2024

확인이 필요한 사항

  • static 위치는 괜찮은가?
  • 환경변수에 따른 static 경로는 괜찮은가?
  • 보조 함수를 서비스 파일에 너무 많이 만들었는데 괜찮은가?

수정해야할 사항

  • otl 프론트의 시간표 색상과 일치하지 않음 -> 디자인팀에서 정확한 16가지 색상을 받아서 수정 필요

  • 프론트에서 테스트 작동안함 -> 프론트 백엔드 포트 문제, 배포시는 문제없을 것으로 예상

  • semester, lecture service repository 분리해야함 -> lecture service의 함수들 중 prisma service 이용하는 함수들을 repository로 분리해야함, share에서 semester로 일부 함수 이동해야함.

  • 영어시간표 줄간격 깨짐 -> drawTile 함수 수정 필요

  • 글자 출력시 미묘한 픽셀차이 존재 -> drawTile 함수 수정 필요

  • ESLint 통과못함 -> ITimetable.ts 등의 파일 수정 필요

  • 내 시간표 추출 불가능 -> getTimetableEntries 수정 필요

@pbc1017
Copy link
Contributor Author

pbc1017 commented Mar 13, 2024

남은 태스크

  • lecture service의 함수들 중 prisma service 이용하는 함수들을 repository로 분리
  • drawTile 함수 세줄만 출력하도록 수정 필요
  • 내 시간표 추출 불가능 -> getTimetableEntries 수정 필요

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 a pull request may close this issue.

1 participant