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.

Sign up for GitHub

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

Jump to bottom

feat : 엑셀 파일로 과거 시간표 추가하기 #1153

Merged
merged 17 commits into from
Dec 30, 2024
Merged

feat : 엑셀 파일로 과거 시간표 추가하기 #1153

merged 17 commits into from
Dec 30, 2024

Conversation

duehee
Copy link
Contributor

@duehee duehee commented Dec 27, 2024

🔥 연관 이슈

🚀 작업 내용

  1. 엑셀 파일을 이용해 과거 시간표를 추가할 수 있는 API를 작성했습니다.
  • 엑셀 파일이라 함은, 학사에서 제공하는 성적 확인 엑셀입니다.(아래 사진 참조)
image
  • 이를 이용하여 2019학년도 2학기부터 시작하여 정보를 집어넣을 수 있습니다.
  • 현재 재수강과 S/U 중 U를 받게 되면 입력되지 않도록 했습니다. (F의 경우는 인정이 되는지 모르겠어서, 데이터가 있다면 의견 주시면 감사하겠습니다.)
  • 학사에서 제공하는 엑셀 파일의 형식은 "xls"이어서, "xlsx"에 대한 처리는 추가적으로 진행하지 않았습니다.
  • 아래의 사진은 성적표를 집어넣을 시, 데이터가 어떻게 추가되는지에 대한 사항입니다.
  • course_type은 임의로 "전필" 하나를 집어넣고 테스트했는데, 문제 없었습니다. 추후 course_type 크롤링 사항에 따라 상황을 보고 코드 수정 예정입니다.
image image

💬 리뷰 중점사항

엄청 오래 걸릴 사항은 아니었는데, 생각보다 따질 사항이 많아서 코드 작성이 오래 걸린 것 같습니다.
예전 코드를 최신화하며 graduation 패키지에서 model 밖에 없길래 따로 추가해서 작성했습니다.
일단 먼저 리뷰해주시고, 아마 충돌 문제 생길 거 같아서 추가 할 사항 있으면 그것도 말씀해주시면 감사하겠습니다.

브랜치에서 테스트를 하고 싶으신 분이 있다면 아래의 사항을 따라서 진행해주세요.

  1. Swagger에서는 파일이 안 올라가므로, Postman으로 진행합니다.
  2. 로그인을 진행해서 토큰 받고,localhost:8080/graduation/excel/upload 을 POST로 처리합니다.
  3. body의 file을 선택하고, Key에 "file"이라고 적습니다. 그 후 Value에 엑셀 파일을 넣습니다.
image
  1. 아래 사진이 뜬다면 성공! DB에서 확인해보세요.
image

코드 좀 지저분할수도 있습니다. 수정사항 있다면 말씀해주세요. 긴 글 읽어주셔서 감사합니다.😄

@duehee duehee added the Team User 유저 팀에서 작업할 이슈입니다 label Dec 27, 2024
@duehee duehee self-assigned this Dec 27, 2024
@github-actions github-actions bot added the 기능 새로운 기능을 개발합니다. label Dec 27, 2024
Soundbar91
Soundbar91 requested changes Dec 27, 2024
View reviewed changes
Copy link
Contributor

@Soundbar91 Soundbar91 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 💯
졸학계 컨텍스트 부족 + 처음보는 라이브러리 공부할 게 많아 보이네요..!
코멘트 확인 부탁드립니다 👍

src/main/java/in/koreatech/koin/domain/graduation/dto/ExcelStudentData.java Outdated Show resolved Hide resolved
src/main/java/in/koreatech/koin/domain/graduation/service/GraduationService.java Outdated Show resolved Hide resolved
src/main/java/in/koreatech/koin/domain/graduation/service/GraduationService.java Outdated Show resolved Hide resolved
src/main/java/in/koreatech/koin/domain/graduation/service/GraduationService.java Outdated Show resolved Hide resolved
src/main/java/in/koreatech/koin/domain/graduation/service/GraduationService.java Show resolved Hide resolved
src/main/java/in/koreatech/koin/domain/graduation/service/GraduationService.java Outdated
Comment on lines 110 to 112
.name("졸업학점 계산용 테이블")
.isDeleted(false)
.isMain(true)
Copy link
Contributor

Choose a reason for hiding this comment

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

R

이 부분도 컨텍스트가 부족해서 여쭙니다..!

이렇게 되면 졸업학점 계산용 테이블의 이름으로 사용자 시간표가 생기게 되는데, 사용자가 이 이름을 꺼려할 수 있을 것 같다는 생각이 듭니다..

메인 시간표만 학점 계산에 들어간다고 기억이 나는데, 관련해서 회의 때 이야기 나누면 좋을 것 같아요 !

Copy link
Contributor Author

Choose a reason for hiding this comment

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

이름을 어떻게 지으면 좋을까요? 그냥 고민고민하다가 일단 저 이름으로 넣어놓긴 했는데, 의견 있다면 부탁드려요 :D

Copy link
Contributor

Choose a reason for hiding this comment

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

메인 시간표..? 어렵네요 네이밍은 역시 😢

src/main/java/in/koreatech/koin/domain/graduation/service/GraduationService.java Outdated Show resolved Hide resolved
src/main/java/in/koreatech/koin/domain/graduation/service/GraduationService.java Show resolved Hide resolved
src/main/java/in/koreatech/koin/domain/graduation/dto/ExcelStudentData.java Outdated Show resolved Hide resolved
duehee and others added 13 commits December 29, 2024 12:35
@duehee
chore : 개행 추가
0ca55f8
@Soundbar91 @duehee
feat: lecture 컬럼 크기 수정 flyway 추가 (#1142)
1132002
@dradnats1012 @duehee
fix: 식단 이미지 알림 롤백 (#1145)
03b1d7f 
* fix: 스케줄러 cron식 수정

* fix: 식단 이미지 업로드 알림 롤백
@Choon0414 @duehee
fix : 동계방학 생협 운영시간 업데이트 (#1144)
0025f00 
* fix : 생협 동계학기 운영 시간 수정

* fix : 생협 동계학기 운영 시간 수정

* chore : 공백 추가

* fix: 세탁소 전화번호 수정
@Soundbar91 @duehee
feat: 스웨거 그룹화 (#1138)
1026bd4 
* chore: swagger 패키지 생성 및 클래스 이동

* feat: 비즈니스팀 API 그룹화

* chore: 비즈니스 API 추가

* feat: 캠퍼스팀 API 그룹화

* feat: 유저팀 API 그룹화

* feat: ABTEST API 그룹화

* feat: BCSD API 그룹화

* chore: 비즈니스팀 API 추가

* feat: 어드민 API 그룹화

* chore: 유저팀 API 추가

* feat: 로그인 API 그룹화 및 그룹 이름 변경

* refactor: 패키지 경로 enum화

* refactor: 중복 코드 메소드화

* refactor: 그룹 스웨거 파일 분할

* chore: 미사용 import 삭제

* chore: 메소드 명 변경

* chore: 리뷰 반영
@Choon0414 @duehee
fix: 안경점, 우편취급국 운영시간 추가, 대즐 운영시간 오타 수정 (#1151)
2974f48
@duehee
feat: flyway추가 & model 관련 추가 및 수정
3c50708
@Soundbar91 @duehee
feat: 시간표 조회 API 이수 구분 반환 추가 (#1150)
0e40eca 
* feat: repository 추가

* feat: exception 추가

* feat: 이수 구분 반환값 추가

* feat: coursetype 반환값 추가

* test: 테스트 코드 수정

* chore: 리뷰 반영
@Soundbar91 @duehee
fix: department 중복 수정
913e9cb
@Soundbar91 @duehee
fix: department 조회 로직 변경
13cea01
@duehee
rebase : 충돌 해결
5c15785
@duehee
chore : 현수님 리뷰 반영
5881418
@duehee
Merge branch 'feature/1140-graduation-credit-calculator' into feature…
2bd768f 
…/new-add-graduation-excel
@duehee duehee merged commit 03ad206 into feature/1140-graduation-credit-calculator Dec 30, 2024
2 checks passed
@duehee duehee deleted the feature/new-add-graduation-excel branch December 30, 2024 13:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@20HyeonsuLee 20HyeonsuLee 20HyeonsuLee left review comments

@Soundbar91 Soundbar91 Soundbar91 approved these changes

@seongjae6751 seongjae6751 Awaiting requested review from seongjae6751

@daheeParkk daheeParkk Awaiting requested review from daheeParkk

@kwoo28 kwoo28 Awaiting requested review from kwoo28

Assignees

@duehee duehee

Labels
Team User 유저 팀에서 작업할 이슈입니다 기능 새로운 기능을 개발합니다.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@duehee @Soundbar91 @20HyeonsuLee @dradnats1012 @Choon0414