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

[Main] V2.2024.11.17.01 #202

Merged
merged 5 commits into from
Nov 17, 2024
Merged

[Main] V2.2024.11.17.01 #202

merged 5 commits into from
Nov 17, 2024

Conversation

Youthhing
Copy link
Member

No description provided.

gikhoon and others added 5 commits November 7, 2024 02:13
* refactor: Extract AttendanceMemberInfo record from AttendanceRecordResponse

* feat: Add record for single attendance record response

* refactor: change attendance record statistic class name

AttendanceRecordResponse
-> AttendanceRecordStatisticResponse

* feat: Generate single attendance record result service

* fix: fix counting absent record of statistic

* Revert "fix: fix counting absent record of statistic"

This reverts commit aa39e7b.

* refactor: change statistic response name

AttendanceRecordStatisticResponse -> GenerationMemberAttendanceRecordResponse

* refactor: change one attendance record response name

SingleAttendanceRecordResponse
-> AttendanceRecordResponse
* chore: 엑셀 추출을 위한 의존성 추가

* feat: 출석 정보 엑셀 추출 구현

* style: 불필요한 세미콜 삭제 및 코드 컨벤션에 맞춰서 스타일 수정

* refactor: 코드 가독성 향상을 위한 메서드 분리

* refactor: 기존에 정의된 enum (AttendanceType, AttendanceResult)을 활용하여 출석 상태 처리

* chore: api path 수정 및 securityConfig에 추가

* style: 불필요한 세미콜론 삭제 및 코드 컨벤션에 맞춰서 스타일 수정

* refactor: 코드 가독성 향상을 위한 메서드 분리

* refactor: 기존에 정의된 enum (AttendanceType, AttendanceResult)을 활용하여 출석 상태 처리

* chore: api path 수정 및 securityConfig에 추가

* refactor: Service 단의 ResponseEntity(byte[]) 반환을 byte[]로 변경 및 Controller 수정

* feat: 파일 다운로드 시 동적 파일명 지정 기능 추가

- Excel 파일 다운로드 API에서 기본 파일명을 동적으로 설정할 수 있도록 fileName 파라미터 추가
- 한국어 파일명 지원을 위해 파일명 인코딩 처리 로직 추가

* refactor: URLEncoder 인코딩 방식 StandardCharsets.UTF_8로 수정

* refactor: 불필요한 증가 방지

* refactor: 최초에 한번만 활동중인 멤버를 가져오게 수정

* refactor: 매직넘버 대신 기존 enum 필드 활용

* style: 코드 컨벤션에 형식 맞추기 및 기본 파일 이름 변경

* refactor: '부원 이름' 매직넘버 대신 기존 enum 필드 활용

* refactor: 확장성을 위해 sheet 이름 따로 설정하지 않게 수정

* style: controller 반환 형식 수정

* refactor: 출석 기록을 먼저 가져온 이후에 '결석'으로 채우는 걸로 로직 변경

* refactor: 동명이인 이슈를 고려해서 member id를 가지고 매핑

* refactor: 세션에 맞게 자동으로 엑셀 이름 추출되어서 다운되도록 수정

* refactor: sessionId 가져오는 로직 수정

* style: 메소드명 수정

* refactor: 출석 기록 엑셀 파일 생성 및 데이터를 입력하는 메소드 AttendanceExcelUtil로 이동

* refactor: 출석 정보 수집 및 엑셀 파일 생성 방식 개선 (중복 조회 최소화 및 동시성 문제 해결)

* chore: MemberRepository에서 사용되지 않는 `findById` 메서드 삭제

* refactor: updateAttendanceRecords 메서드에서 단일 Attendance 객체 처리 방식으로 개선

* style: 변수명 오타 수정

* style: 열과 행의 순서를 나타내는 변수 네이밍 변경

* refactor: memberStatisticsMap 키 정렬 작업을 반복문 외부로 이동

* refactor: 출석 정보를 조회한 이후 세션 정보 반환하게 수정

* fix: 활동 부원 조회 시 발생할 수 있는 동시성 이슈 해결

* style: 공백 삭제

* fix: controller에서 출결id는 가져오게 수정

* fix: 출석 정보를 못 가져오는 에러 수정

* refactor: getAttendanceRecords 기능 분리

* refactor: AttendanceExcelUtil의 createExcelFile 메서드에서 회원 정보를 주입받도록 변경

* style: 사용하지 않는 private field 삭제

* refactor: 출석 데이터 준비 로직을 util에서 service로 이전

* style: 메서드 네이밍 수정

* style:불필요한 else if 제거하여 코드 간결화

* refactor: 코드 유연성을 위해 변수 타입 수정

* refactor: Excel 파일 다운로드를 위한 HttpHeaders 설정을 유틸리티 클래스로 분리

* refactor: 엑셀 다운로드 헤더 설정 로직을 AttendanceExcelHeaderUtil로 분리하여 호출 방식 개선

* chore: 사용되지 않는 파일 인코딩 에러 삭제

* style: 변수명을 명확하게 수정하여 가독성 향상

* refactor: entrySet을 사용하여 키와 값을 동시에 가져오도록 수정

* refactor: 배열 대신 List<Integer> 사용으로 컬렉션 프레임워크 적용 및 코드 일관성 개선

* style: 주석 제거

* style: 변수 네이밍 수정

* refactor: 프라이빗 기본 생성자 삭제

* style: map 관련 변수명 수정

* 리팩터: 출석 횟수 표현을 Map<String, Integer>로 변경

* feat: defaultValue 종류 나눠서 상수화

* style: map 관련 변수명 수정

* fix: resolve merge conflict

---------

Co-authored-by: Youth <[email protected]>
Co-authored-by: youth <[email protected]>
[Release] V2.2024.11.16.01
@Youthhing Youthhing merged commit cc64f87 into main Nov 17, 2024
1 check passed
@Youthhing Youthhing deleted the release branch November 17, 2024 14:30
@Youthhing Youthhing restored the release branch November 19, 2024 10:54
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 this pull request may close these issues.

3 participants