Skip to content

feat: ABEEK 영역의 상세보기 모달창 기능 구현 #86

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
Merged
merged 31 commits into from
Nov 18, 2024
Merged

feat: ABEEK 영역의 상세보기 모달창 기능 구현 #86

merged 31 commits into from
Nov 18, 2024

Conversation

@whxogus215
Copy link
Contributor

@whxogus215 whxogus215 commented Nov 17, 2024 •
edited
Loading

🔧연결된 이슈

🛠️작업 내용

🤷‍♂️PR이 필요한 이유

  • 상세보기 버튼을 눌렀을 때, 해당 영역에 대한 테이블을 보여줍니다.
  • 처음에 인증정보 확인을 눌렀을 때, 불러와지는 statsForm.html과 abeekDetailModal.html이 함께 렌더링됩니다.
    • 다만, 상세보기 버튼을 눌렀을 때만 모달창이 떠야하므로 렌더링만 되고, 브라우저에 표시되지는 않습니다. (display : none)

image

  • 만약, 전문교양 영역의 상세보기 버튼을 누르면, 해당 값이 인자로 넘어가 자바 스크립트 코드에서 해당 타입의 tbody만 표시합니다.

image

  • MSC 영역의 상세보기 버튼을 눌렀을 때 보이는 화면

✔️PR 체크리스트

  • 필요한 테스트를 작성했는가?
  • 다른 코드를 깨뜨리지 않았는가?
  • 연결된 이슈 외에 다른 이슈를 해결한 코드가 담겨있는가?

@whxogus215 whxogus215 added 🔧 기능 추가 새로운 기능 추가 디자인 웹 화면 관련 이슈 labels Nov 17, 2024
@whxogus215 whxogus215 requested a review from gmltn9233 November 17, 2024 07:51
@whxogus215 whxogus215 self-assigned this Nov 17, 2024
whxogus215 and others added 18 commits November 17, 2024 16:54
@whxogus215
style: 불필요한 줄바꿈 제거
df7c98f
@gmltn9233
refactor: CompletedCoursesService
b5d4bd3 
기준을 벗어난 학수번호일 경우 저장하지 않음
@gmltn9233
Merge pull request #87 from Sejong-Java-Study/fix/CompletedCourse
d16f6b6 
fix: 기이수 과목 학수번호 오류 수정
@gmltn9233
refactor: GonghakCoursesByMajorDto
d14fd64 
GonghakCoursesByMajorDto 를 CourseDetailsDto로 변경하고 이번 modal 기능에 필요한 semester 필드만 추가하였습니다.
@gmltn9233
refactor:GonghakCoursesDao
675e790 
GonghakCoursesByMajorDto 를 리팩토링함에 따라 semester도 함께 반환하도록 리팩토링하였습니다.
@gmltn9233
refactor: dto 구조 변경
3d3b5d0 
최종적으로 controller로 반환되는 GongakResultDto는 AbeekTypeConst 별 AbeekDetailsDto를 가지고 있고 AbeekDetailsDto는 이수 학점 정보인 ResultPoint와 기이수 과목 정보 리스트 coursesDetails 을 가지고 있습니다.
@gmltn9233
refactor: Test
26166da 
dto 네이밍 변경에 따른 테스트 코드 리팩토링
@gmltn9233
refactor: GonghakCalculateService
9c816c7 
메서드 분리, CourseDetails 로직 추가
@gmltn9233
feat: GonghakCalculateServiceTes
3b1d3b9 
AbeekTypeConst 별로 CourseDetailsDto가 올바르게 저장되는지 확인하는 테스트 코드
@gmltn9233
refactor: StatusController
a68ebde 
Service로부터 userResultRatioDto를 받아온 후 분리하여 각각 resultPoint, coursesDetail로 모델에 전달
@whxogus215
refactor: tr, td 태그 들여쓰기 1 depth 추가
7271830 
tr 태그가 thead, tbody 태그와 같은 depth를 갖고 있어서 가독성 차원에서 들여쓰기를 추가하였습니다.
@whxogus215
feat: 타임리프 네임스페이스 추가
81e8fbc
@whxogus215
feat: 모달창 뼈대 구성 및 자세히 버튼 추가
1f616c4
@whxogus215
feat: 모달창 클릭 시 열기/닫기 트리거 연결
915f923
@whxogus215
feat: 모달창, 배경 CSS 적용
8f05527
@whxogus215
refactor: html 파일 이름 첫 글자 소문자로 변경
9272c8e
@whxogus215
refactor: 각 영역별 이수한 과목 정보가 tbody 단위로 생성되도록 타임리프 수정
5e854cf
@whxogus215
feat: 특정 타입의 상세보기를 눌렀을 때, 해당 타입의 테이블만 표시되도록 구현
94308d5
@whxogus215
style: 불필요한 줄바꿈 제거
118de86
@whxogus215
fix: 모달창 컨테이너 크기 고정
05213f2 
상세영역이 많을 경우, 스크롤할 수 있도록 모달창 컨테이너의 div 영역의 크기를 고정하였습니다.
@whxogus215
fix: 이수한 과목의 년도를 불러오도록 수정
4f1fd15
@whxogus215
refactor: layout 템플릿 이름 변경
d518299 
- layout -> mainLayout : main 화면에서 쓰이는 레이아웃
- layout2 -> userLayout : 로그인 후 쓰이는 레이아웃
@whxogus215
merge fix
abe676e
@whxogus215 whxogus215 linked an issue Nov 18, 2024 that may be closed by this pull request
영역별 인증정보를 확인하는 기능을 추가한다 #83
Closed
@gmltn9233 gmltn9233 merged commit e22ac7c into develop Nov 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gmltn9233 gmltn9233 gmltn9233 approved these changes

Assignees

@whxogus215 whxogus215

Labels

🔧 기능 추가 새로운 기능 추가 디자인 웹 화면 관련 이슈

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

영역별 인증정보를 확인하는 기능을 추가한다

3 participants

@whxogus215 @gmltn9233