feat : 회원 정보 수정 by duehee · Pull Request #236 · BCSDLab/KOIN_API_V2

duehee · 2024-03-23T12:38:02Z

🔥 연관 이슈

close PUT /user/student/me #23

🚀 작업 내용

회원 정보 수정 추가했습니다.

기존 Swagger에서 사용하던 Request 유형에서 Password를 제외하고 작성했습니다.(다른 이슈로 할당 예정)
기존에 사용하던 학번/학부 예외처리를 422번이 아닌 ~~404~~번 오류 -> 400번 오류로 작성했습니다.(3/27 수정)
학번에 관한 오류는 현재 10자리가 아니면 오류가 발생하도록 했습니다.(23년 이후 학번처리에 대한 의견이 필요)

~~2. 학부/학과에 대한 ENUM 추가했습니다.~~
~~- 24년 기준, 나뉜 학부도 추가해서 작성했습니다.~~
(현재 존재하는 학부/과 : 기계공학부, 컴퓨터공학부, 메카트로닉스공학부, 전기전자통신공학부, 디자인공학부, 건축공학부, 화학생명공학부, 에너지신소재공학부, 산업경영학부, 고용서비스정책학과)
~~- 기존 존재하던 TEST의 내용도 수정 완료했습니다.~~

-> ENUM을 사용하는 방향을 조금 고민해봐야 할 것 같아서 STRING으로 우선 작성했습니다.
서버 내에서 500번 오류 발생 이슈로 수정했고, 추가적으로 이슈 생기면 스레드에 작성하겠습니다!
StudentDepartment ENUM은 삭제하지 않고 유지해놨습니다. -> 검증 ENUM에서 진행

💬 리뷰 중점사항

기능 작성하는 시간이 조금 오래 걸렸습니다. 기존에 회원 정보 수정에 있던 내용인 비밀번호 관련이슈는 User 팀의 스레드에 올려놓았습니다.

작성하며 헤맨 부분이 많아서, 리뷰 작성해주시면 감사하겠습니다! 감사합니다 :D

… feature/23-put-user-student-me

github-actions · 2024-03-23T12:51:35Z

Unit Test Results

98 tests 98 ✔️ 40s ⏱️
17 suites   0 💤
17 files   0 ❌

Results for commit fe6adfb.

♻️ This comment has been updated with latest results.

kwoo28

기존 코인 보면서 해석하고 코마거치는 과정이 쉽진 않았을텐데 수고 많으셨습니다!
리뷰 남기겠습니다.

kwoo28 · 2024-03-23T17:11:12Z

src/main/java/in/koreatech/koin/domain/user/model/User.java

+        }
+        if (user.resetExpiredAt != null) {
+            this.resetExpiredAt = user.resetExpiredAt;
+        }


C

StudentService에서 받는 필드는 gender, name, nickname, phoneNumber인데, 왜 모든 필드에 조건문을 달으신건가요??

기존 코인 API에서 update 메소드를 이용했는데, 이 구문에는 모든 값에 처리가 되어있어서 같이 작성해보았습니다!

수정이 필요한 필드가 있다면 해당 필드(들)만 수정하는 메서드를 작성하는 게 좋을 것 같아요~
사이드이펙트가 발생할 여지가 많아 보입니다.

추가적으로 수정해서 작성했습니다~

R

메소드 명만 봤을때는 null이 들어왔을 때 null로 update되는게 맞아보이는데 이거는 updateOptional... 같은 느낌이네요
굳이 null이면 업데이트를 안한다는 로직을 넣어야할 이유가 있나요?

오히려 이 검증이 있어서 메소드가 예상하지 못하는 동작을 수행하게 되는거같네요

아예 모르는 사람이 이 메소드를 사용했을 때 인자로 들어가는 모든 값으로 수정이 일어나길 원할텐데 한번 고민해보세요

+) 현재 업데이트 대상인 필드들이 엔티티 조건에서 NotNull이 다 걸려있나요?

현재 업데이트 대상인 필드들은 NN이 걸려있지 않습니다!(DB)
확인하여 업데이트 수정해보았습니다! 확인해주시면 감사하겠습니당

kwoo28 · 2024-03-24T06:06:59Z

src/main/java/in/koreatech/koin/domain/user/service/StudentService.java

+            .name(studentUpdateRequest.name())
+            .nickname(studentUpdateRequest.nickname())
+            .phoneNumber(studentUpdateRequest.phoneNumber())
+            .build();


C

필드4개를 변경시키고자 User객체를 한개 더만드신건가요? User클래스에서 변경시키고자 하는 필드만 받는 메서드를 만드는게 어떨까요?

User 객체에서 @Bulider를 이미 사용하고 있는 메소드가 있어서 추가적인 @Builder를 작성하지 않았습니다.
제가 기존에 사용하던 방법인 새 update 객체를 생성하여 적용하는 방법이 효율이 좋지 않은가에 대해선 한 번 생각해보겠습니다 :D

개인적으로 Builder를 타 객체 정보 수정을 위한 임시 객체 생성에 사용하는 것은 선호하지 않습니다. user.update 메서드에 파라미터로 User를 주는 것보다 수정이 필요한 각각의 변수를 할당하는 것이 훨씬 가독성에 좋지 않을까 싶습니다.
(user.update에 User 객체를 통째로 주면 호출자 입장에서는 어떤 필드가 사용될지 예측할 수 없습니다)

kwoo28 · 2024-03-24T06:07:47Z

src/main/java/in/koreatech/koin/domain/user/service/StudentService.java

+        Student updateStudent = Student.builder()
+            .department(StudentDepartment.from(studentUpdateRequest.major()))
+            .studentNumber(studentUpdateRequest.studentNumber())
+            .build();


C

이것도 User와 마찬가지로...

위의 사항과 동일하여 한 번 생각해보겠습니다!!

kwoo28 · 2024-03-24T06:11:25Z

src/main/java/in/koreatech/koin/domain/user/service/StudentService.java

+
+        if (studentUpdateRequest.studentNumber() != null &&
+            !Student.isValidStudentNumber(studentUpdateRequest.studentNumber())) {
+            throw new StudentNumberNotValidException(


C

이 부분은 예외를 만들어서 처리하기보다 Request에서 @Valid로 학번의 형식을 거르는 방법은 어떤가요?

그렇게 코드를 작성한다면 더 깔끔하게 처리할 수도 있겠네요!

이거 반영이 안된거같아요~

반영하여 원래 있던 예외처리 구문을 삭제했습니다!

seongjae6751

고생하셨습니다! 리뷰 드립니다

seongjae6751 · 2024-03-24T10:09:19Z

src/main/java/in/koreatech/koin/domain/user/dto/StudentUpdateRequest.java

+        @Size(max = 50, message = "학번은 50자 이내여야 합니다.")
+        @Schema(description = "학번", example = "2020136065")
+        String studentNumber


C
원경님이 말씀하신대로 학번을 50자로 받고 10자인지 확인하는 함수로 확인하는 것보다
처음부터 10자인지 확인해주면 좋을 것 같아요!

@Size( max = 10, message = "학번은 정확히 10자 여야 합니다.") String studentNumber;

이런 식으로 쓸 수 있을 것 같네요!

기존 레거시 코드에는 학번이 50으로 되어있어서 이렇게 작성해보았는데, 확인해보고 수정 해보겠습니다!

seongjae6751 · 2024-03-24T10:57:06Z

src/test/java/in/koreatech/koin/acceptance/UserApiTest.java

+        String token = jwtProvider.createToken(student.getUser());
+
+        ExtractableResponse<Response> response = RestAssured
+            .given().log().all()


R
.log().all()은 빼주세요~

seongjae6751 · 2024-03-24T11:12:52Z

src/main/java/in/koreatech/koin/domain/user/controller/UserApi.java

+            @ApiResponse(responseCode = "403", content = @Content(schema = @Schema(hidden = true))),
+            @ApiResponse(responseCode = "404", content = @Content(schema = @Schema(hidden = true))),
+            @ApiResponse(responseCode = "409", content = @Content(schema = @Schema(hidden = true))),
+            @ApiResponse(responseCode = "422", content = @Content(schema = @Schema(hidden = true)))


A
사소한 거지만 422 안 쓰기로 했으면 빼도 될 것 같네요~

songsunkook · 2024-03-25T05:12:56Z

src/main/java/in/koreatech/koin/domain/user/model/User.java

+    public void update(User user) {
+        if (user.nickname != null) {
+            this.nickname = user.nickname;
+        }
+        if (user.name != null) {
+            this.name = user.name;
+        }
+        if (user.phoneNumber != null) {
+            this.phoneNumber = user.phoneNumber;
+        }
+        if (user.gender != null) {
+            this.gender = user.gender;
+        }
+    }


R

네이밍이 직관적이지 못한 것 같아요
User를 매개변수로 받았을 때 해당 변수에서 어떤 내용을 update해주는 것인지 알 수 없을 것 같습니다.
좀 더 리팩토링이 필요해 보여요~

+) Student도 수정해주세요

각 이름 userInfoUpdate, studentInfoUpdate로 수정 완료했습니다!

songsunkook

테스트 꼼꼼하게 작성해주셨네요!
코멘트 남겨드렸으니 확인 부탁드립니다~

songsunkook · 2024-03-25T23:58:09Z

src/main/java/in/koreatech/koin/domain/user/dto/StudentUpdateRequest.java

+        @Schema(description = "[NOT UPDATE]졸업 여부(true, false)", example = "false")
+        Boolean isGraduated,
+
+        @Schema(description = "전공{기계공학부, 컴퓨터공학부, 메카트로닉스공학부, 전기전자통신공학부, 디자인공학부, 건축공학부, 화학생명공학부, 에너지신소재공학부, 산업경영학부, 고용서비스정책학과}", example = "컴퓨터공학부")


C

저희 코딩 컨벤션에 한 줄당 최대 길이가 정해져있지 않았나요??
중간에 개행을 넣을 수 있을 것 같아요

songsunkook · 2024-03-26T00:07:06Z

src/main/java/in/koreatech/koin/domain/user/dto/StudentUpdateResponse.java

+    @Schema(description = "성별(남:0, 여:1)", example = "1")
+    Integer gender,
+
+    @Schema(description = "전공{기계공학부, 컴퓨터공학부, 메카트로닉스공학부, 전기전자통신공학부, 디자인공학부, 건축공학부, 화학생명공학부, 에너지신소재공학부, 산업경영학부, 고용서비스정책학과}", example = "컴퓨터공학부")


C

저희 코딩 컨벤션에 한 줄당 최대 길이가 정해져있지 않았나요??
중간에 개행을 넣을 수 있을 것 같아요

22

songsunkook · 2024-03-26T00:09:50Z

src/main/java/in/koreatech/koin/domain/user/exception/StudentDepartmentNotValidException.java

+
+import in.koreatech.koin.global.exception.DataNotFoundException;
+
+public class StudentDepartmentNotValidException extends DataNotFoundException {


A

StudentDepartmentNotValidException 보다는 StudentDepartmentNotFoundException가 더 적절한 네이밍일 것 같아요

형식이 맞지 않을때에 예외를 처리하는 것이어서 Valid를 사용했는데, 이름을 짓는다면 예외에 대한 이름을 짓는 게 맞는지, 아니면 여기서는 반환하는 값이 404여서 Not Found를 쓰는 게 맞는지 궁금합니다!
--> 형식이 맞지 않는 오류여서 예외처리는 Valid가 맞는 것 같습니다!

형식이 맞지 않는 오류라면 DataNotFoundException을 상속하는게 옳은 방향일까요~?

songsunkook · 2024-03-26T00:10:40Z

src/main/java/in/koreatech/koin/domain/user/exception/StudentNumberNotValidException.java

+
+import in.koreatech.koin.global.exception.DataNotFoundException;
+
+public class StudentNumberNotValidException extends DataNotFoundException {


A

StudentDepartmentNotValidException 보다는 StudentDepartmentNotFoundException가 더 적절한 네이밍일 것 같아요

22

songsunkook · 2024-03-26T00:14:59Z

src/main/java/in/koreatech/koin/domain/user/service/StudentService.java

+            .name(studentUpdateRequest.name())
+            .nickname(studentUpdateRequest.nickname())
+            .phoneNumber(studentUpdateRequest.phoneNumber())
+            .build();


개인적으로 Builder를 타 객체 정보 수정을 위한 임시 객체 생성에 사용하는 것은 선호하지 않습니다. user.update 메서드에 파라미터로 User를 주는 것보다 수정이 필요한 각각의 변수를 할당하는 것이 훨씬 가독성에 좋지 않을까 싶습니다.
(user.update에 User 객체를 통째로 주면 호출자 입장에서는 어떤 필드가 사용될지 예측할 수 없습니다)

songsunkook · 2024-03-26T00:19:03Z

src/test/java/in/koreatech/koin/acceptance/UserApiTest.java

+    }
+
+    @Test
+    @DisplayName("학생이 정보를 수정한다 - 이미 있는 닉네임이라면 409")


A

이걸 빨리 적용했어야 했는데 ;-;

Choi-JJunho

고생하셨습니다~
코멘트 몇개 확인부탁할게요

Choi-JJunho · 2024-03-27T02:30:52Z

src/main/java/in/koreatech/koin/domain/user/dto/StudentResponse.java

            user.getEmail(),
            user.getGender().name(),
-            student.getDepartment(),
+            student.getDepartment().toString(),


C

toString, getValue가 혼용되어 사용되고있는 것 같아요~

Suggested change

student.getDepartment().toString(),

student.getDepartment().getValue(),

Department 관련해서 추가하며 다른 요소들을 너무 신경쓰지 않은 것 같네요..! :<

Choi-JJunho · 2024-03-27T02:31:44Z

src/main/java/in/koreatech/koin/domain/user/exception/StudentDepartmentNotValidException.java

+
+import in.koreatech.koin.global.exception.DataNotFoundException;
+
+public class StudentDepartmentNotValidException extends DataNotFoundException {


형식이 맞지 않는 오류라면 DataNotFoundException을 상속하는게 옳은 방향일까요~?

Choi-JJunho · 2024-03-27T02:34:10Z

src/main/java/in/koreatech/koin/domain/user/exception/StudentNumberNotValidException.java

+        String message = String.format("%s %s", DEFAULT_MESSAGE, detail);
+        return new StudentNumberNotValidException(message);
+    }
+


R

불필요한 공백 발견~

Choi-JJunho · 2024-03-27T02:40:39Z

src/main/java/in/koreatech/koin/domain/user/service/StudentService.java

+                "학생의 학번 형식이 아닙니다. studentNumber : " + studentUpdateRequest.studentNumber());
+        }
+
+        user.userInfoUpdate(studentUpdateRequest.nickname(), studentUpdateRequest.name(),


R

info라는 네이밍이 다소 추상적이게 다가올 수 있어요
그냥 update는 어떨까요?

Suggested change

user.userInfoUpdate(studentUpdateRequest.nickname(), studentUpdateRequest.name(),

user.update(studentUpdateRequest.nickname(), studentUpdateRequest.name(),

참고자료: https://github.com/Yooii-Studios/Clean-Code/blob/master/Chapter%2002%20-%20%EC%9D%98%EB%AF%B8%20%EC%9E%88%EB%8A%94%20%EC%9D%B4%EB%A6%84.md#%EC%9D%98%EB%AF%B8-%EC%9E%88%EA%B2%8C-%EA%B5%AC%EB%B6%84%ED%95%98%EB%9D%BC%EB%B6%88%EC%9A%A9%EC%96%B4-noise-word-%EB%A5%BC-%EC%93%B0%EC%A7%80-%EB%A7%90%EC%9E%90

그렇다면 패스워드를 수정하는 메소드가 추가된다면, 그 메소드 이름을 따로 설정하는 것이 더 나을 수 있을 거 같네요!

Choi-JJunho · 2024-03-27T02:41:25Z

src/main/java/in/koreatech/koin/domain/user/service/StudentService.java

+
+        user.userInfoUpdate(studentUpdateRequest.nickname(), studentUpdateRequest.name(),
+            studentUpdateRequest.phoneNumber(), UserGender.from(studentUpdateRequest.gender()));
+        student.studentInfoUpdate(studentUpdateRequest.studentNumber(),


R

info라는 네이밍이 다소 추상적이게 다가올 수 있어요
그냥 update는 어떨까요?

ditto

Choi-JJunho · 2024-03-27T02:44:08Z

src/test/java/in/koreatech/koin/acceptance/UserApiTest.java

                    .isEqualTo(user.getGender().name());
                softly.assertThat(response.body().jsonPath().getString("major"))
-                    .isEqualTo(student.getDepartment());
+                    .isEqualTo(student.getDepartment().name());


R

name이랑 같아야하는게 맞아요? getValue 랑 같아야하는거 아닌가??

앗.. StudentDepartment ENUM 추가 이후 수정하지 않은 것 같습니다!

Choi-JJunho

고생하셨습니다~
Approve의 길은 멀고도 험하죠 ㅋㅋㅋ
코멘트 확인 부탁드릴게요~

Choi-JJunho · 2024-03-27T05:52:51Z

src/main/java/in/koreatech/koin/domain/user/controller/UserApi.java

-    @ApiResponses(
-        value = {
-            @ApiResponse(responseCode = "200"),
-            @ApiResponse(responseCode = "400", content = @Content(schema = @Schema(hidden = true))),
-            @ApiResponse(responseCode = "401", content = @Content(schema = @Schema(hidden = true))),
-            @ApiResponse(responseCode = "403", content = @Content(schema = @Schema(hidden = true))),
-            @ApiResponse(responseCode = "404", content = @Content(schema = @Schema(hidden = true))),
-        }
-    )
-    @Operation(summary = "푸쉬알림 동의 여부 조회")
-    @GetMapping("/user/notification")
-    ResponseEntity<NotificationStatusResponse> checkNotificationStatus(
-        @Auth(permit = {STUDENT, OWNER}) Long memberId
-    );
-
-    @ApiResponses(
-        value = {
-            @ApiResponse(responseCode = "200"),
-            @ApiResponse(responseCode = "400"),
-            @ApiResponse(responseCode = "401"),
-            @ApiResponse(responseCode = "403"),
-            @ApiResponse(responseCode = "404"),
-        }
-    )
-    @Operation(summary = "푸쉬알림 동의")
-    @PostMapping("/user/notification")
-    ResponseEntity<Void> permitNotification(


R

안돼~~ 알림기능 다 어디갔어

바로 다시 추가했씁니다..

Choi-JJunho · 2024-03-27T05:53:24Z

src/main/java/in/koreatech/koin/domain/user/exception/StudentDepartmentNotValidException.java

+public class StudentDepartmentNotValidException extends IllegalArgumentException {
+    private static final String DEFAULT_MESSAGE = "학생의 전공 형식이 아닙니다.";


R

개행

Suggested change

public class StudentDepartmentNotValidException extends IllegalArgumentException {

private static final String DEFAULT_MESSAGE = "학생의 전공 형식이 아닙니다.";

public class StudentDepartmentNotValidException extends IllegalArgumentException {

private static final String DEFAULT_MESSAGE = "학생의 전공 형식이 아닙니다.";

Choi-JJunho · 2024-03-27T05:53:33Z

src/main/java/in/koreatech/koin/domain/user/exception/StudentNumberNotValidException.java

+public class StudentNumberNotValidException extends IllegalArgumentException {
+    private static final String DEFAULT_MESSAGE = "학생의 학번 형식이 아닙니다.";


R

여기도 개행

Choi-JJunho · 2024-03-27T05:53:49Z

src/main/java/in/koreatech/koin/domain/user/model/StudentDepartment.java

+    CHEMICAL("화학생명공학부"),
+    ENERGY("에너지신소재공학부"),
+    INDUSTRIAL("산업경영학부"),
+    EMPLOYMENT("고용서비스정책학과");


R

후행쉼표

Suggested change

EMPLOYMENT("고용서비스정책학과");

EMPLOYMENT("고용서비스정책학과"),

;

Choi-JJunho · 2024-03-27T05:54:18Z

src/main/java/in/koreatech/koin/domain/user/model/StudentDepartment.java

+        return Arrays.stream(values())
+            .filter(it -> it.value.equals(value))
+            .findAny()
+            .orElseThrow(() -> new StudentDepartmentNotValidException("학생의 전공 형식이 아닙니다. " + value));


C

withDetail 메소드를 선언 & 활용하지 않은 이유가 있나요

사용해서 작성해보았습니다!!

Choi-JJunho · 2024-03-27T05:58:11Z

src/main/java/in/koreatech/koin/domain/user/model/User.java

+        }
+        if (user.resetExpiredAt != null) {
+            this.resetExpiredAt = user.resetExpiredAt;
+        }


R

메소드 명만 봤을때는 null이 들어왔을 때 null로 update되는게 맞아보이는데 이거는 updateOptional... 같은 느낌이네요
굳이 null이면 업데이트를 안한다는 로직을 넣어야할 이유가 있나요?

오히려 이 검증이 있어서 메소드가 예상하지 못하는 동작을 수행하게 되는거같네요

아예 모르는 사람이 이 메소드를 사용했을 때 인자로 들어가는 모든 값으로 수정이 일어나길 원할텐데 한번 고민해보세요

+) 현재 업데이트 대상인 필드들이 엔티티 조건에서 NotNull이 다 걸려있나요?

Choi-JJunho · 2024-03-27T05:58:51Z

src/main/java/in/koreatech/koin/domain/user/model/UserGender.java

+            return Arrays.stream(values())
+                .filter(it -> it.ordinal() == index)
+                .findAny()
+                .orElseThrow(() -> new IllegalArgumentException("잘못된 성별 인덱스 입니다. index : " + index));


C

어떤 친구는 customException으로 notfound를 반환하고 어떤 친구는 IAE를 발생시키는 기준이 있나요?

제가 학번과 학부에 대한 예외처리는 custumException을 만들어서 처리했는데, 성별에 관한 사항은 그냥 적용시킨 것 같습니다!
성별에 관한 예외처리에 대한 상황은 한 번 고려해보는 것이 좋을 것 같습니다😄

이 사항은 추가적으로 custumException 작성해보았습니다!

Choi-JJunho · 2024-03-27T05:59:43Z

src/main/java/in/koreatech/koin/domain/user/model/UserGender.java

+                .findAny()
+                .orElseThrow(() -> new IllegalArgumentException("잘못된 성별 인덱스 입니다. index : " + index));
+        }
+        return null;


R

대상 index가 없으면 예외가 터지는게 맞는거같은데 어떻게 생각하나요
서비스 내에서 null이 자유분방하게 돌아다니면 어딘가에서는 NPE가 발생할 것 같네요

null 처리 구문 삭제했습니다!

Choi-JJunho · 2024-03-27T06:00:06Z

src/main/java/in/koreatech/koin/domain/user/service/StudentService.java

+
+        if (studentUpdateRequest.studentNumber() != null &&
+            !Student.isValidStudentNumber(studentUpdateRequest.studentNumber())) {
+            throw new StudentNumberNotValidException(


이거 반영이 안된거같아요~

Choi-JJunho · 2024-03-27T06:01:40Z

src/main/java/in/koreatech/koin/domain/user/dto/StudentUpdateRequest.java

+        @Size(max = 10, message = "학번은 10자여야 합니다.")
+        @Schema(description = "학번", example = "2020136065")
+        String studentNumber


R

아래 코멘트에서 나온 내용을 여기에 적용시켜보면 될 것 같아요

https://github.com/BCSDLab/KOIN_API_V2/pull/236/files#r1536745436

추가하여 적용 완료 했습니다!

songsunkook

거의 다 온 것 같은데 조금만 더 힘내봅시다..!
화이팅~

songsunkook · 2024-03-28T11:47:51Z

src/main/java/in/koreatech/koin/domain/user/model/Student.java

-    @Size(max = 50)
    @Column(name = "major", length = 50)
-    private String department;
+    @Enumerated(EnumType.STRING)
+    private StudentDepartment department;


R

major에서 department로 컬럼명을 다르게 매핑한 이유가 있나요? DB와 헷갈릴 것 같아요

+) 중요한 내용인데, 이거 잘 실행되나요? 에러 납니다. 푸시 전에는 로컬에서 반드시 실행해보시기 바랍니다.

기존에 작성되어있던 Student의 major 매핑의 컬럼명이 department여서 동일하게 Enum 생성하는 과정에서 이름 사용했습니다!

songsunkook · 2024-03-28T11:54:51Z

src/main/java/in/koreatech/koin/domain/user/service/StudentService.java

+        if (studentUpdateRequest.nickname() != null &&
+            userRepository.existsByNickname(studentUpdateRequest.nickname())) {
+            throw DuplicationNicknameException.withDetail("nickname : " + studentUpdateRequest.nickname());
+        }


C

짜잘한 내용이지만 닉네임 검증 로직은 메서드를 분리하는 것도 좋아 보이네요

따로 Service 내 메소드를 작성하도록 하겠습니다!!!

Choi-JJunho

고생하셨습니다~
작은 컨벤션들만 확인 부탁드릴게요

Choi-JJunho · 2024-03-28T09:54:37Z

src/main/java/in/koreatech/koin/domain/user/dto/StudentUpdateResponse.java

+    @Schema(description = "익명 닉네임", example = "익명_1676688416361")
+    String anonymousNickname,
+
+    @Schema(description = "이메일 주소 \n", example = "koin123@koreatech.ac.kr")


R

불필요한 내용같네요

Suggested change

@Schema(description = "이메일 주소 \n", example = "koin123@koreatech.ac.kr")

@Schema(description = "이메일 주소", example = "koin123@koreatech.ac.kr")

Choi-JJunho · 2024-03-29T03:23:44Z

src/main/java/in/koreatech/koin/domain/user/dto/StudentUpdateResponse.java

+        + "건축공학부, 화학생명공학부, 에너지신소재공학부, 산업경영학부, 고용서비스정책학과}", example = "컴퓨터공학부")
+    String major,
+
+    @Schema(description = "이름 \n", example = "최준호")


R

불필요한 내용같네요

ditto

Suggested change

@Schema(description = "이름 \n", example = "최준호")

@Schema(description = "이름", example = "최준호")

Choi-JJunho · 2024-03-29T03:26:01Z

src/main/java/in/koreatech/koin/domain/user/service/StudentService.java

+        Student student = studentRepository.getById(userId);
+        User user = student.getUser();
+
+        if (studentUpdateRequest.nickname() != null &&
+            userRepository.existsByNickname(studentUpdateRequest.nickname())) {
+            throw DuplicationNicknameException.withDetail("nickname : " + studentUpdateRequest.nickname());
+        }
+
+        user.update(studentUpdateRequest.nickname(), studentUpdateRequest.name(),
+            studentUpdateRequest.phoneNumber(), UserGender.from(studentUpdateRequest.gender()));
+        student.update(studentUpdateRequest.studentNumber(),
+            StudentDepartment.from(studentUpdateRequest.major()));
+
+        studentRepository.save(student);
+


A

개인 취향입니다만 메소드 내부에 있는 친구들은 붙여줘도 좋지 않을까용

Suggested change

Student student = studentRepository.getById(userId);

User user = student.getUser();

if (studentUpdateRequest.nickname() != null &&

userRepository.existsByNickname(studentUpdateRequest.nickname())) {

throw DuplicationNicknameException.withDetail("nickname : " + studentUpdateRequest.nickname());

}

user.update(studentUpdateRequest.nickname(), studentUpdateRequest.name(),

studentUpdateRequest.phoneNumber(), UserGender.from(studentUpdateRequest.gender()));

student.update(studentUpdateRequest.studentNumber(),

StudentDepartment.from(studentUpdateRequest.major()));

studentRepository.save(student);

Student student = studentRepository.getById(userId);

User user = student.getUser();

if (studentUpdateRequest.nickname() != null &&

userRepository.existsByNickname(studentUpdateRequest.nickname())) {

throw DuplicationNicknameException.withDetail("nickname : " + studentUpdateRequest.nickname());

}

user.update(studentUpdateRequest.nickname(), studentUpdateRequest.name(),

studentUpdateRequest.phoneNumber(), UserGender.from(studentUpdateRequest.gender()));

student.update(studentUpdateRequest.studentNumber(),

StudentDepartment.from(studentUpdateRequest.major()));

studentRepository.save(student);

저는 주로 작동하는 단위로 끊는 편인데, 코드 내에서는 스타일 통일 해보겠습니당

Choi-JJunho · 2024-03-29T03:27:20Z

src/test/java/in/koreatech/koin/acceptance/UserApiTest.java

+        Student student = Student.builder()
+            .studentNumber("2019136135")
+            .anonymousNickname("익명")
+            .department(StudentDepartment.COMPUTER)
+            .userIdentity(UserIdentity.UNDERGRADUATE)
+            .isGraduated(false)
+            .user(
+                User.builder()
+                    .password("1234")
+                    .nickname("주노")
+                    .name("최준호")
+                    .phoneNumber("010-1234-5678")
+                    .userType(STUDENT)
+                    .gender(UserGender.MAN)
+                    .email("test@koreatech.ac.kr")
+                    .isAuthed(true)
+                    .isDeleted(false)
+                    .build()
+            )
+            .build();
+
+        studentRepository.save(student);


A

어짜피 인증에서 거르는 테스트면 저장 안해도되지않을까요

테스트 상단의 get /user/student/me 401번 오류 검증 과정과 동일하게 구성해보았습니다!

duehee · 2024-03-29T16:28:55Z

이번에 로컬 환경에서 확인하면서 작동해본 결과, 오류가 많이 발생해서 추가한 사항들입니다!
gender, department 등 현재 enum으로 작동하고 있는 코드들에 null 값을 허용하도록 메소드를 조금 수정했습니다.
(.ordinal() 사용 시 intValue() 호출로 인한 Npe 발생 및 department.getValue() 사용 간 동일한 npe 발생)
무분별한 null 사용은 문제가 될 수 있지만, DB 상으로 null 값이 문제가 없는 사항들이라 판단하여 추가했습니다.
코드가 조금 지저분한 점은 있지만, 감안하여 작성해봤습니다. 어떻게 수정하면 좋을지 의견 있으시다면 달아주세요!

현재 ENUM을 사용하여 "컴퓨터공학부"를 저장하면 COMPUTER로 DB에 저장되고 있는데, 혹시 이렇게 저장되는 방식이 맞는 지 궁금해서 이렇게 코멘트 남깁니다! 이게 맞다면 더 이상 로컬에서의 문제는 발생하지 않을 것 같아서, 의견 부탁드립니다.

songsunkook

자세한 내용은 DM으로 설명해드렸으니 좀 더 고민해보고 알려주세요
아직 500에러 날겁니다

songsunkook · 2024-03-30T06:00:56Z

src/main/java/in/koreatech/koin/domain/user/service/StudentService.java

+    public void CheckDepartmentValid(String department) {
+        List<String> departments = Arrays.asList(
+            "컴퓨터공학부",
+            "기계공학부",
+            "메카트로닉스공학부",
+            "전기전자통신공학부",
+            "디자인공학부",
+            "건축공학부",
+            "화학생명공학부",
+            "에너지신소재공학부",
+            "산업경영학부",
+            "고용서비스정책학과"
+        );
+
+        if (department != null & !departments.contains(department)) {
+            throw StudentDepartmentNotValidException.withDetail("학부(학과) : " + department);
+        }
+    }


R

이렇게 검증이 필요하다면 차라리 enum 클래스로 빼서 검증하는 건 어떨까요?

수정 완료했습니다!!!!!

songsunkook

코멘트 70개의 전설..
정말 고생하셨습니다 ! 👍

@RequestParam

* test: TestContainer 적용 (#27) * test: TestContainer 적용 * test: TestContainer MySQL 버전 5.7로 변경 * test: RestAssured port 선언조건 추가 * feat: POST /user/login API 구현 (#50) * feat: User Entity 추가 * test: /user/login 인수테스트 작성 * feat: 로그인 뼈대코드 구성 * feat: testcontainer 도입 * test: TestContainer 적용 * test: TestContainer MySQL 버전 5.7로 변경 * test: RestAssured port 선언조건 추가 * feat: 사용자 로그인 구현 * feat: 사용자 마지막 로그인한 날짜 추가 * test: ActiveProfiles 설정 추가 * build: 토큰 값 minute 주석 추가 * refactor: refreshToken 만료일자 14일로 수정 * build: 불필요한 설정파일 제거 * refactor: 예외처리부 추가 * test: 학교 메일로 테스트케이스 수정 * refactor: 반환값 accessToken임을 명시 * feat: 상점 메뉴 조회 API 구현 (#112) * refactor: 전체 패키지 구조 변경 (#114) * refactor: 패키지 구조 변경 * refactor: DTO 레코드로 변경 * refactor: 상점 도메인 패키지 구조 변경 * refactor: 패키지 표현 변경 * feat: GET /lands API 구현 (#148) * feat : 도메인 생성 * faet : controller 구현 * feat : response DTO 생성 * feat : 전체 조회 service 구현 * feat : 전체 조회 repository 생성 * feat : Domain 생성 * test : 인수 테스트 작성 * refactor : 하위 패키지 중 domain을 model로 수정 * refactor : DTO 클래스에 JsonNaming 어노테이션 사용 * refactor : Column 어노테이션 안에 필요없는 내용 제거 * refactor : 복덕방 리스트 조회 메서드에 Transaction(readOnly=true)가 적용되도록 수정 * refactor : 리스트 원소로 사용하던 LandResponse의 이름을 LandListItemResponse로 수정 * feat: 사용자 인증 구현 (#149) * feat: 사용자 인증 구현 * refactor: ArgumentResolver 이름 변경 * refactor: CustomException 적용 * refactor: 예외상황 명시하도록 순서 변경 * refactor: 상수추출 * refactor: 미사용 생성자 제거 * chore: 패키지 변경 * refactor: 미사용 어노테이션 제거 * refactor: 오타수정 * refactor: 오류메시지 구체화 * refactor: 미사용 메서드 제거 * �feat: GET /dept, GET /depts 구현 (#147) * refactor: 패키지 구조 변경 * refactor: DTO 레코드로 변경 * refactor: 상점 도메인 패키지 구조 변경 * feat: 학과 조회 관련 틀 작성 * feat: 학과명 열거형 추가 * feat: 학과 조회 초안 작성 * feat: 학과 조회 정상 동작 * refactor: 열거형 제거 최대한 도입해보려 시도했으나 `@Enumerated(EnumType.STRING)`으로는 불가능한 구조인 것으로 판단하여 제거 (DB에 한글로 저장되어 있어서 enum과 매핑하려면 한글 열거형이 필요해보임) ERROR DETAIL: "No enum constant in.koreatech.koin.domain.dept.domain.DeptType.기계공학부" * rename: 정적 팩토리 메서드 네이밍 수정 * test: 인수 테스트 추가 * fix: 정상 동작하도록 수정 (복합키 매핑 이전) * refactor: 복합키 매핑 * remove: 미사용 코드 제거 * feat: GET /depts 구현 * test: GET /depts 테스트 작성 * rename: 도메인명 수정 Dept -> DeptInfo * fix: DeptInfoRepository Key 수정 * refactor: service 코드 간소화 * fix: GET /dept 응답 메시지 자료형 수정 * refactor: 트랜잭션 적용 * rename: 변수명 수정 * refactor: 응답 객체 자료형 수정 * refactor: 개행 컨벤션 준수 * rename: DTO 네이밍 수정 * refactor: 개행 컨벤션 준수 * refactor: 메서드 네이밍 수정 * test: 테스트 인자 상수화 * test: 코드 간소화 * fix: Repository ID 제네릭 수정 * refactor: Static Import로 변경 Response 클래스들에 붙은 SnakeCaseStrategy.class를 static import하여 가독성 향상 * test: GET /depts 테스트 코드 수정 1. 기존에 연관관계를 테스트하지 않던 코드 수정 2. 반복문을 제거하고 일일이 매핑하도록 수정 * refactor: DB에서 enum으로 구조 변경 cc. #152 * rename: 메서드명 컨벤션 준수 * rename: 컨트롤러 메서드명 컨벤션 준수 * refactor: 존재하지 않을 시 응답 코드 변경 * style: enum 개행 조정 * remove: DB 미사용으로 인한 불필요 코드 제거 --------- Co-authored-by: Invidam <hansupark0208@gmail.com> * feat: logout 구현 (#154) * feat: accessToken 재발급 (#155) * feat: logout 구현 * feat: accessToken 재발급 구현 * fix: 응답 수정 및 테스트 수정 * refactor: 사용자 인증 요구사항 제거 * refactor: 리프레시토큰 userId 구분 * �feat: 게시글 목록 조회 구현 (#158) * feat: 골격 작성 * feat: 엔티티 작성 * feat: repository 작성 * feat: service 작성 * feat: response 작성 * feat: DB 조회 로직 작성 * feat: Jsoup 의존성 추가 * feat: 응답 내용 추가 1. article.is_notice 2. article.contentSummary 3. board.is_notice 4. board.children (must null) * feat: 예외처리 작성 * refactor: 페이징 관련 정보 클래스 분리 * fix: soft delete된 게시판 or 게시글은 조회 안되도록 수정 * fix: 1페이지 조회 안되는 버그 수정 * fix: 누락된 응답 추가 (summary) * fix: 최신글부터 조회하도록 수정 * test: 테스트 코드 작성 * move: ErrorResponseWrapper 위치 변경 * refactor: 코딩 컨벤션 준수 * refactor: 문자열 파싱 메서드 리팩토링 * test: 예외상황 테스트 추가 * feat: 메뉴 카테고리 목록 조회 (#153) * feat: 메뉴 카테고리 조회 기능 구현 * refactor: 응답 DTO 생성 로직 수정 * refactor: cascade 처리 방식 변경 refactor: cascade 처리 방식 변경 * test: 테스트 문맥 변경 * feat: 복덕방 단일 조회 (#156) * feat : controller 구현 * feat : repository에 id로 조회 기능 생성 * feat : service에 조회 기능 구현 * feat : 응답 DTO 구현 * refactor : domain 필드 자료형 변경 * refactor : stream 라인 변경 * refactor : 코드 축약 * test : 단일 조회 테스트 작성 * refactor : 에러 메시지 구체적으로 수정 * refactor : 기존 응답 구조와 동일하도록 수정 * test : 테스트에 이미지 추가 * refactor:� findByXXX -> getByXXX (#164) * refactor: default method 활용한 getByXXX 메소드 추가 * style: 공벡 제거 * refactor: 복덕방에도 적용 * refactor: DataNotFoundException으로 묶어서 관리 * feat: 게시글 단일 조회 (#173) * feat: 골격 작성 * feat: Repository 메서드 추가 * feat: 응답 객체 작성 * feat: Comment 도메인 엔티티 작성 * feat: 초안 완성 * fix: 로그인된 사용자 정보 습득 절차 수정 1. 비로그인 유저의 요청도 받도록 수정 2. 올바르지 않은 토큰 입력 시 권한 없음으로 진행되도록 수정 * rename: 응답 객체 변수명 수정 * refactor: 내용 요약 정보 최신화 시기 변경 기존: getter 호출 시마다 로직 수행하여 추출 변경: jpa 엔티티 로드 후 1회 로직 수행 * refactor: 유저 인증 정보 전달 방식 수정 기존: 컨트롤러단에서 @RequestHeader로 받아 전달 변경: 서비스단에서 RequestContextHolder로 습득 * feat: 게시글 조회 기록 저장 및 조회수 변동 구현 유저의 게시글 조회가 조회수 증가로 이어지는 간격을 1시간으로 설정 * fix: 게시글 조회 관련 버그 수정 기존에 게시글 조회 이력이 남아있는데 만료 시간이 지난 경우, 기존 내역을 업데이트해줘야 하나 새로 저장을 시도하여 구문 에러가 발생함. 위 상황에는 기존 내역을 업데이트하는 로직을 추가함으로써 문제 해결 * fix: 게시글 목록 조회 버그 수정 문제: 최대 페이지 초과 시 404 에러가 발생하지 않음. 원인: 예외처리 로직을 별도 함수로 분리했으나 기존 함수를 사용중이었음 해결: 예외처리가 적용된 함수로 호출문 변경 * fix: 조회 기록 Builder에 만료시간 기입 * test: 신규 테스트 추가 및 기존 테스트 수정 * other: 학과 조회 자료형 수정 학과 코드: Long -> String * refactor: 코드스타일 개선 * rename: 메서드 네이밍 수정 * refactor: 빈 게시글 목록 조회 예외처리 제거 * refactor: IP 주소 습득 로직 분리 * test: 예외 제거에 따른 테스트 수정 * refactor: 연관관계 방식으로 로직 및 테스트 수정 * refactor: ArticleViewLog의 FK를 연관관계로 매핑 * refactor: Auth 어노테이션 활용하도록 변경 (#175) * refactor: Auth 어노테이션 활용하도록 변경 * refactor: IPAddress Interceptor 추가 * chore: auth global 패키지로 이동 * style: 로깅 디테일 * refactor: 불필요한 UserType 제거 * docs: add swagger config (#176) * refactor: Auth 어노테이션 활용하도록 변경 * refactor: IPAddress Interceptor 추가 * docs: swagger 추가 * chore: auth global 패키지로 이동 * style: 로깅 디테일 * refactor: 불필요한 UserType 제거 * refactor: Parameter Path 수정 * feat: 버전 단일 조회 (#168) * feat: request dto 작성 * feat: 엔티티 작성 * feat: response dto 작성 * feat: 서비스 작성 * feat: 컨트롤러 작성 * feature: 기존 코드 제거 및 테스트 코드 추가 * feat: 리뷰 적용 * feat: VersionType 피드백 적용 및 Exception Test Case 추가 * feat: 공백 컨벤션 수정 & No newline at end of file 적용 * feat: 회원 탈퇴 구현 (#178) * feat: 회원 탈퇴 구현 * refactor: 줄바꿈 * feat: 인기 게시글 목록 조회 (#181) * feat: API 골격 작성 * feat: Redis 접근 기능 구 * feat: 만료 체크 로직 자동화 * feat: 인기 많은 게시글 출력 로직 작성 1. 모든 로그 조회 2. 조회수 내림차순 정렬 3. Limit로 자르기 4. 결과 반환 * refactor: 스케줄러 관심사 분리 * feat: 최근 조회된 게시글 수가 10개 미만이면 최신 게시글과 병합 * fix: 응답 객체 필드명 수정 * test: 테스트 작성 * refactor: 조회수 검증 로직 개선 * docs: 응답 객체 설명 추가 * test: 불필요한 콘솔 출력 제거 * refactor: redis 대신 mysql을 사용하도록 수정 * test: 테스트 수정 * feat: 사장님 정보 조회 (#182) * feat: 도메인 엔티티 생성 * feat: 응답 객체 초안 작성 * feat: 도메인 클래스 작성 * feat: API 작성 1. 경로 수정(다른 API로 착각하고 작업중이었어서 수정) 2. 연관관계 매핑 3. API 로직 작성 * test: 테스트 작성 * refactor: 접근제어자 수정 * docs: 응답 객체 문서 주석 추가 * refactor: 트랜잭션 추가 * refactor: 예외처리 보완 * refactor: 양방향 연관관계 제거 * refactor: 잘못 기입된 Setter 제거 * refactor: 기본생성자 접근제어 추가 * refactor: 소프트딜리트된 대상은 제외 * refactor: 기본생성자 접근제어 추가 * style: 컨벤션 준수 * rename: 메서드명 변경 * feat: 사장님 자신의 모든 상점 조회 (#183) * feat: 도메인 엔티티 생성 * feat: 응답 객체 초안 작성 * feat: 도메인 클래스 작성 * feat: API 작성 1. 경로 수정(다른 API로 착각하고 작업중이었어서 수정) 2. 연관관계 매핑 3. API 로직 작성 * test: 테스트 작성 * feat: API 작성 * test: 테스트 작성 * refactor: 트랜잭션 적용 * docs: 응답 객체 문서 주석 추가 * fix: 버그 수정 * refactor: 읽기 전용 트랜잭션으로 수정 * refactor: 개행 컨벤션 준수 * feat: 사장님 회원가입 인증번호 전송 요청 (#184) * feat : owner controller 구현 * refactor: 기존 url에 맞게 수정 * feat: 이메일 관련 도메인 생성 & 검증 기능 구현 * feat: 중복 이메일인지 검증하는 기능 구현 * feat : AwsSesConfig 작성 * feat : SesMailSender 작성 * feat: 메일폼 작성 & 템플릿 가져오는 기능 구현 * feat: 메일 보내는 기능 구현 * feat: 메일 보낼 수 있는지 검사 * feat: OwnerInVerification 작성 * feat: redis에 저장 * fix: 타임리프 가져오는 에러 해결 * fix: 메일 전송 에러 해결 * refactor: mail form loader 설정 코드 메서드로 분리 * fix: 중복 이메일 검증 오류 해결 * feat: slack에 알림 보내는 기능 구현 * feat: 스웨거 작성 * refactor: 예외처리 수정 * feat: 슬랙 test url 작성 * feat: 레디스에 있는 사장님 정보 조회 구현 * test: 회원가입 인증번호 전송 요청 테스트 작성 * style: 클래스&메서드명 수정 * style: global의 commaon 패키지를 domain 패키지로 수정 * test: 테스트 메서드명 수정 * refactor: 인증번호 랜덤 생성을 따로 구현 * refactor: Aws SES 서울 리전으로 수정 * refactor: 이메일 검증 로직 수정 * fix: AWS SES 기존 Region으로 수정 * refactor: 스프링 이벤트 적용해서 슬랙 알림 전송하도록 수정 * refactor: 메일폼 가져오는 라이브러리 thymeleaf로 변경 * test: 슬래 전송 실패해도 200으로 응답하는 테스트로 수정 * test: 사장님 회원가입 인증번호 전송 요청 이벤트 발생 시 슬랙 전송 이벤트가 발생하는지 확인하는 테스트 * refactor: EmailAddress 객체 정적 팩토리 메서드를 사용하도록 수정 * refactor: OwnerInVerification 팩토리 메서드명 수정 * refactor: noticeEmailVerification()에서 코드 줄임 * refactor: 사용하지 않는 코드 제거 * test: MockBean을 SpyBean으로 수정 * test: MockBean 위치 변경 * test: application test yml에 thymeleaf 경로 설정 추가 * test: test/resources/mail/owner메일폼 추가 * test: test/resources/owner메일폼 제거 * test: MockBean 위치 AcceptanceTest로 변경 * style: record 변수 선언 세로로 위치 변경 * style: OwnerInVerification의 AccessLevel static import로 수정 * style: User 생성자 인수 일정하게 나열 * refactor: User 생성자 private으로 수정 * style: 모든 AccessLevel을 static import로 수정 * style: Owner, OwnerAttachment Doamin 위치 domain 패키지에서 model패키지로 수정 * refactor: aws ses 최신 버전으로 수정 * fix: 버전 매핑 에러 수정 (#191) * fix: 버그 수정 * refactor: 예외 수정 * test: 테스트 분리 * feat: 버전 최신화 메서드 작성 * refactor: 응답 간결화 및 버그 수정 * feat: 이메일 중복 체크 (#194) * feat: 이메일 중복 체크 구현 * feat: 이메일 중복 테스트 구현 * chore: application-test.yml 수정 - datasource 추가 * chore: application-test.yml 수정 - datasource 삭제 * refactor: ModelAttribute 적용 - @RequestParam -> @ModelAttribute * feat: 이메일 중복 테스트 추가 - 이메일을 보내지 않으면 400 - 잘못된 이메일 형식이면 400 * style: 라인 포맷팅 * feat : 강의 목록 조회(학기별) (#192) * feat : Lecture 클래스 생성 * feat : LectureApi 클래스 생성 * feat : LectureController 클래스 생성 * feat : LectureRepository 클래스 생성 * feat : LectureResponse DTO 생성 * feat : LectureService 클래스 생성 * feat : LectureApiTest 테스트 생성 후 통과 * refactor : 리뷰 반영 * refactor : classTime 반환타입 Integer[]로 변경 * refactor : 리뷰 반영 * refactor : Integer[]배열 List<Long>으로 변경 * refactor : TimeTable로 네이밍 변경 * refactor : 리뷰 반영 * feat: 버스 남은 시간 조회 - 셔틀버스, 등하교 버스 (#185) * feat: API 골격 작성 * feat: mongoDB 의존성 추가 * feat: 응답 객체 작성 * feat: 예외 추가 * feat: 엔티티 작성 * feat: 미운행인 버스 필터링 * feat: enum 추가 * feat: 남은 시간 계산 로직 작성 * feat: 가장 최근 버스까지 남은 시간 응답 구현 * feat: 미운행 요일 제외 로직 추가 * feat: 버스 정거장 검증로직 적용 및 중복되는 시간 제외 * refactor: enum 객체를 사용하도록 수정 * refactor: 등하교 방향 기준 탐색에서 정류장 탐색으로 로직 변경 * remove: 미사용 코드 제거 BusCourse.isRunning() * refactor: 테스트 mocking 위해 Clock 사용하도록 수정 * test: 테스트컨테이너 mongoDB 추가 * test: 셔틀 버스 조회 테스트 추가 * docs: API 문서에 파라미터 정보 명시 * remove: 불필요 주석 제거 * refactor: 누락된 Transactional 추가 * fix: mocking 변수 선언부 이동 * remove: 불필요 주석 제거 * refactor: 도착 시각 패턴 불일치 시 예외처리 * refactor: 반복문을 stream으로 변환 * refactor: 엔티티 필드로 List 사용 시 미리 초기화 * refactor: String 비교 시 NPE 예방 * refactor: 불필요한 매핑 메서드 제거 * refactor: 변수 분리 * rename: 변수명 수정 * refactor: Builder 생성자 접근제어자 수정 * refactor: enum 필드 간소화 * fix: 미사용 예외 사용하도록 수정 * feat: 출발지와 도착지가 일치할 경우 예외 처리 * refactor: IllegalArgumentException 예외 응답 수정 * feat : 학기 조회(전학기) (#196) * feat : Semester get API 구현 완료 * chore : 리뷰 반영 코드 위치 수정 / 패키지 이름 변경 * style : 라인 포매팅 및 클래스 삭제 * test : 테스트 코드 수정 * feat : 사장님 식당 생성, 특정 상점 조회 API구현 (#199) * feat: 상점 생성 기능 추가 * feat: 상점등록을 하기 위해 연관된 model정의 * feat: shop과 관련된 repository, exception작성 * feat: 1차 테스트 완료 * feat: 사장님 상점 생성 기능 구현 * feat: 특정상점조회 API구현 * refactor: 특정상점조회 테스트코드 수정 * chore: import경로 오류 수정 * chore: pull충돌 해결 * refactor: saveAll제거 * feat: 스웨거 설명 추가 * refactor: Builder생성자 접근제어 변경 * refactor: ShopResponse 적팩메이름 of로 변경 * refactor: ShopResponse 적팩메이름 of로 변경 * feat: 닉네임 중복 체크 (#189) * refactor: 회원 탈퇴 반환타입 수정 * feat: 중복 데이터 예외 처리 추가 * feat: 닉네임 중복 예외 처리 추가 * feat: 닉네임 중복 체크 구현 * feat: 닉네임 중복 테스트 * refactor: 예외 처리 수정 * refactor: 닉네임 중복 로직 수정 * feat: 닉네임 중복 중복아닐때, 제약조건위반시 테스트 추가 * refactor: 파라미터 valid 적용 * refactor: 전역 데이터 중복 예외 제거(이미 전에 누가 만들었음) * feat: 충돌 해결 * refactor: 코멘트 반영 수정 * feat : 식단 Api 구현 (#193) * feat : 충돌 해결 * chore : VersionException 제거 * chore : SnakeCaseStrategy import static 수정 * refactor : 엔티티 id int -> Long 수정 * feat : 사장님 식당 생성, 특정 상점 조회 API구현 (#199) * feat: 상점 생성 기능 추가 * feat: 상점등록을 하기 위해 연관된 model정의 * feat: shop과 관련된 repository, exception작성 * feat: 1차 테스트 완료 * feat: 사장님 상점 생성 기능 구현 * feat: 특정상점조회 API구현 * refactor: 특정상점조회 테스트코드 수정 * chore: import경로 오류 수정 * chore: pull충돌 해결 * refactor: saveAll제거 * feat: 스웨거 설명 추가 * refactor: Builder생성자 접근제어 변경 * refactor: ShopResponse 적팩메이름 of로 변경 * refactor: ShopResponse 적팩메이름 of로 변경 * feat : 커스텀 예외처리 추가 * chore : 충돌 해결 * chore : 테스트 정보 수정 * chore : 파라미터 설명 추가 * refactor : menu String -> List 수정 * fix : 테이블명 수정 * refactor : 현재시간에 대한 테스트를 위해 clock 모킹 추가 * remove : 사용하지 않는 예외 삭제 * chore : 테스트코드 public 제외, 불필요한 공백 삭제 --------- Co-authored-by: Hyeonsu Lee <127578418+20HyeonsuLee@users.noreply.github.com> * feat : 특정상점 전체메뉴조회 api작성 (#208) * feat: 특정 상점 모든 메뉴조회 api 작성 * feat: 특정 상점 모든 메뉴조회 api 테스트코드 작성 * feat: 코드리뷰사항 반영 1 차 * feat: 코드 리뷰사항 반영 2차 * feat : BCSD Lab 활동 api구현 (#207) * feat: bcsd lab 활동 가져오기 1차 테스트 통과 * refactor: 반환 값을 리스트에서 맵으로 변경 * refactor: activitiy api 반환값 맵으로 변경 * refactor: controller가 ActivityApi를 구현하게 변경 * chore: 불필요한 문자 제거 * style: 서비스 로직 최적화 * chore: 불필요한 코드 삭제 * chore: 충돌 해결 * chore: 불필요한 문자 제거 * chore: 1차 코드 리뷰 피드백 적용 * chore: 테스트 정보 주정 * style: 테스트에 param 적용 * chore: 줄 바꿈 수정 * style: 반환 타입 을 dto로 변경 * style: 컨트롤러 변수의 반환타입을 명시적으로 변경 * chore: 피드백 반영 * chore: 테스트 코드 내용 추가 * chore: ActivitiesResponseList Dto이름 변경 * feat: Presigned URL 구현 (#210) * feat: Enum Converter 추가 * style: 변수명 변경 * style: 불필요한 주석 제거 * refactor: Controller 테스트 제거 * refactor: 예외 수정 * chore: 패키지 구분 * refactor: 외부 API로 구분 * style: 공백제거 * test: 테스트 수정 * refactor: 변수명 변경 * refactor: fileName 포함하도록 수정 * refactor: request dto swagger 설명 추가 * refactor: passwordEncoder 활용하도록 수정 (#211) * fix: 특정 상점 전체메뉴조회 응답 객체 미스매치 수정 (#214) * feat: 특정 상점 모든 메뉴조회 api 작성 * feat: 특정 상점 모든 메뉴조회 api 테스트코드 작성 * feat: 코드리뷰사항 반영 1 차 * feat: 코드 리뷰사항 반영 2차 * fix: menus응답 수정 * chore: 충돌 해결 * chore: 충돌 해결 * chore: 응답 필드 이름 수정 * chore: 응답 필드 이름 수정 * feat: 시간표 기능 구현 (#212) * feat: 시간표 조회 기능 추가 * feat: 시간표 생성 기능 추가 * feat: 시간표 수정/삭제 기능 추가 * fix: 시간표 수정 변경 * refactor: 리뷰 반영 1차 * refactor: 리뷰 반영 2차 * refactor: 리뷰 반영 3차 * refactor: UpdateTimeTableRequest 이름 수정 UpdateTimeTableRequest -> TimeTableUpdateRequest * refactor: 리뷰 반영 5차 * refactor: 리뷰 반영 6차 * feat: 파일 단건 업로드 구현 (#227) * feat: 파일 단건 업로드 구현 * refactor: swagger 문서 추가 * refactor: swagger 설명 추가 * refactor: 미사용 변수 제거 * refactor: 파일 최대크기 설정 변경 * refactor: 필드주입방싱을 생성자주입 방식으로 변경 * feat: 파일 다중 업로드 구현 (#229) * feat: 파일 다중 업로드 구현 * docs: swagger 수정 * feat: 영양사 권한 추가 (#231) * feat: 영양사 권한 추가 * fix: 후행쉼표 추가 * fix: 복덕방 전제 조회 응답 객체 미스매치 수정 (#215) * chore: LandListItemResponse dto이름 LandsResponse로 변경 * fix: 반환값 list에서 key value형태로 변경 * fix: UploadServiceTest 수정 * style: 라인 포맷팅 * chore: ActivityResponse 예시 수정 * style: dto를 일반 클래스에서 record클래스로 수정 * style: ActivitiesResponse dto를 일반 클래스에서 record클래스로 수정 * feat: FCM 설정 (#232) * refactor: 상점 Entity 연관관계 설정 (#237) * feat: shop연관관계 수정 * refactor: 서비스레이어 dto변환 로직 제거 * feat: 상점사장님이 특정 상점을 조회하는 api작성 (#240) * feat: 상점조회 api작성 * chore: 라인포맷팅 * chore: 권한이 없는 사장님이 특정 상점 조회 테스트 코드 분리 --------- Co-authored-by: HyeonsuLee <leehyeonsu4888@naver.com> * feat : 영양사 품절 정보 입력 (#238) * feat : Dining 클래스에 soldOut 컬럼 추가 * feat : DiningResponse DTO에 soldOut 컬럼 추가 * feat : flyway DB에 sold_out 컬럼 추가 * feat : SoldOutRequest DTO 클래스 추가(메뉴아이디, 품절여부) * feat : CoopDiningController, CoopDiningApi 클래스 추 * feat : CoopDiningService 클래스 추가 * feat : DiningRepository 품절여부 변경 메서드 추가, 테스트용 메서드 추 * feat : 영양사 품절 요청 테스트, 권한 확인 테스트 추 * feat : 기존 테스트 동작하도록 수정 * feat : 리뷰 반 * feat : 리뷰 반영, 테스트 방식 변 * feat : coop 패키지 분 * feat : setSoldOut() 메서드 추 * feat : update 방식을 조회 후 set해주는 방식으로 변 * feat : 사용하지 않는 메서드 삭 * feat : 패키지 변경으로 인한 테스트 변경 * feat : 인가 테스트로 변경 * feat : getById()로 수정 * feat : 개행 추가 * feat : 메뉴가 없는 경우 예외 추가 * refactor: 컨벤션 반영 * refactor: snakecase로 변경 * feat : 영양사 코너별 식단 사진 업로드 (#241) * feat : 충돌 해결 * chore : VersionException 제거 * chore : SnakeCaseStrategy import static 수정 * refactor : 엔티티 id int -> Long 수정 * feat : 사장님 식당 생성, 특정 상점 조회 API구현 (#199) * feat: 상점 생성 기능 추가 * feat: 상점등록을 하기 위해 연관된 model정의 * feat: shop과 관련된 repository, exception작성 * feat: 1차 테스트 완료 * feat: 사장님 상점 생성 기능 구현 * feat: 특정상점조회 API구현 * refactor: 특정상점조회 테스트코드 수정 * chore: import경로 오류 수정 * chore: pull충돌 해결 * refactor: saveAll제거 * feat: 스웨거 설명 추가 * refactor: Builder생성자 접근제어 변경 * refactor: ShopResponse 적팩메이름 of로 변경 * refactor: ShopResponse 적팩메이름 of로 변경 * feat : 커스텀 예외처리 추가 * chore : 충돌 해결 * chore : 테스트 정보 수정 * chore : 파라미터 설명 추가 * refactor : menu String -> List 수정 * fix : 테이블명 수정 * refactor : 현재시간에 대한 테스트를 위해 clock 모킹 추가 * remove : 사용하지 않는 예외 삭제 * chore : 테스트코드 public 제외, 불필요한 공백 삭제 * feat : 영양사 컨트롤러, 서비스 구현 * feat : 식단 이미지 변수 추가 * feat : 영양사 권한 및 영양사 도메인 enum 추가 * feat : image 컬럼 추가 * feat : Image 업로드 요청 DTO 생성 * feat : 영양사님의 사진 업로드 테스트 추가 * feat : 특정 식단의 이미지 업데이트 쿼리 추가 * feat : 쿼리문 수정 * chore : log 제거 * refactor : coop Api 분리 * refactor : image_url로 변수명 수정 * refactor : image_url로 변수명 수정 * chore : menuId로 변수명 구체화 * chore : V$ -> V3 버전 수정 * remove : repository 계층 트랜잭션 제거(service 계층 처리) * feat : 허용되지 않은 권한에 대한 테스트 추가 * refactor : update 수정(setter 이용) * refactor : 테스트 - 허용되지않은 권한 수정 * refactor : getById() 추가 * chore : Optional 검증을 위해 findById() -> getById() 수정 * remove() : DynamicUpdate 제거 * chore : 충돌 해결 * chore : 파일 위치 수정 --------- Co-authored-by: Hyeonsu Lee <127578418+20HyeonsuLee@users.noreply.github.com> * feat: 사장님 회원가입 (#244) * feat: 사장님 회원가입 및 슬랙 알림발송 구현 * refactor: slack 알림발송 로직 수정 * refactor: 사장님 회원가입 로직 구성 * test: 미사용 테스트 제거 * style: 코드 포맷팅 * test: 테스트 수정 * refactor: update flyway * feature: 상점 사장님 관련 모든 GET API를 작성했습니다. (#245) * feat: 상점조회 api작성 * chore: 라인포맷팅 * feat: 점주전용 상점관련 조회 api작성 * feat: 사장님 상점 조회 api get 모두 작성 * chore: 리뷰사항 반영 --------- Co-authored-by: HyeonsuLee <leehyeonsu4888@naver.com> * fix: 게시판 공지사항 판단 로직 수정 (#246) * fix: 공지사항 판단 로직 수정 * fix: 공지사항 판단 로직 재수정 * refactor: 게시판 태그 enum으로 분리 * feat: 사장님 인증코드 검증 (#248) * feat: 사장님 인증번호 인증 구현 * refactor: 로직 수정 및 인증이 완료된 값에 대해 레디스 제거 * test: 테스트 추가 * build: 테이블 복원 * refactor: 인증방식 수정, 사용자 인증없이 사용자 id만 필요한 경우 `@UserId`를 사용하도록 수정 * test: MySQL 테스트 컨테이너 버전 8.0.29로 수정 * refactor: ArgumentResolver Webconfig 등록 * refactor: 파일업로드 임시 토큰에 대한 허용 * test: 테스트 수정 * refactor: 변수 사용 * fix: 전공별 커리큘럼 링크 수정, 새로운 과 추가 (#251) * fix: 전공별 커리큘럼 링크 수정, 새로운 과 추가 * fix: 전공별 커리큘럼 링크 다시 수정 * chore: DeptListItemResponse이름을 DeptsResponse로 변경 * style: 와일드 카드 사용하지 않는 것으로 다시 원래대로 복구 * chore: 주석 설명 변경 * style: 열거형 사용해서 필터 하게 변경 * feat: 사장님 비밀번호 재발급 구현 (#253) * feat: 사장님 인증번호 인증 구현 * refactor: 로직 수정 및 인증이 완료된 값에 대해 레디스 제거 * test: 테스트 추가 * build: 테이블 복원 * refactor: 인증방식 수정, 사용자 인증없이 사용자 id만 필요한 경우 `@UserId`를 사용하도록 수정 * test: MySQL 테스트 컨테이너 버전 8.0.29로 수정 * refactor: ArgumentResolver Webconfig 등록 * refactor: 파일업로드 임시 토큰에 대한 허용 * test: 테스트 수정 * feat: 사장님 비밀번호 찾기 API 구현 * refactor: 이메일 형식 수정 * refactor: 인증 제거 * feat : 회원 정보 수정 (#236) * feat : DTO 추가 * feat : 유저 수정 추가 * test : log.all() 삭제 및 422번 오류 삭제 * chore : update 수정 및 사용하지 않는 Request 객체 삭제 * refactor : update 이름 수정 * refactor : DTO major 라인 포맷팅 * chore : bulider() 미사용 및 update 로직 수 * chore : 리뷰 반영 및 학부/학번 404 -> 400번 오류로 수정 * chore : ApiResponse 수정 * chore : 리뷰 반영 * chore : withDetail 수정 * chore : 리뷰 반영(코드 스타일 및 메소드 분리) * chore : Local 내 회원 정보 조회 오류 확인 추가 * chore : 삼항 연산자 제거 * chore : Response department null 처리 추가 * chore : 변수 수정 및 Gender null 처리 추가 * chore : 변수 수정 및 department null 처리 추가 * chore : DB 학과 한글 저장 * chore : DB 저장 체크(Enumerated 삭제) * chore : DB 저장 체크(Enumerated 삭제 및 model 이름 수정) * chore : ENUM 사용하도록 재수정 * fix : ENUM -> STRING 사용 * chore : 검증 ENUM으로 이동 * chore : ENUM 영어 변수로 수정 * chore : ENUM 후행 쉼표 추가 * fix: 학기 최신 순으로 반환 (#268) * feat : 알림 구독 Api 구현 (#269) * feat : 충돌 해결 * chore : VersionException 제거 * chore : SnakeCaseStrategy import static 수정 * refactor : 엔티티 id int -> Long 수정 * feat : 사장님 식당 생성, 특정 상점 조회 API구현 (#199) * feat: 상점 생성 기능 추가 * feat: 상점등록을 하기 위해 연관된 model정의 * feat: shop과 관련된 repository, exception작성 * feat: 1차 테스트 완료 * feat: 사장님 상점 생성 기능 구현 * feat: 특정상점조회 API구현 * refactor: 특정상점조회 테스트코드 수정 * chore: import경로 오류 수정 * chore: pull충돌 해결 * refactor: saveAll제거 * feat: 스웨거 설명 추가 * refactor: Builder생성자 접근제어 변경 * refactor: ShopResponse 적팩메이름 of로 변경 * refactor: ShopResponse 적팩메이름 of로 변경 * feat : 커스텀 예외처리 추가 * chore : 충돌 해결 * chore : 테스트 정보 수정 * chore : 파라미터 설명 추가 * refactor : menu String -> List 수정 * fix : 테이블명 수정 * refactor : 현재시간에 대한 테스트를 위해 clock 모킹 추가 * remove : 사용하지 않는 예외 삭제 * chore : 테스트코드 public 제외, 불필요한 공백 삭제 * feat : 영양사 컨트롤러, 서비스 구현 * feat : 식단 이미지 변수 추가 * feat : 영양사 권한 및 영양사 도메인 enum 추가 * feat : image 컬럼 추가 * feat : Image 업로드 요청 DTO 생성 * feat : 영양사님의 사진 업로드 테스트 추가 * feat : 특정 식단의 이미지 업데이트 쿼리 추가 * feat : 쿼리문 수정 * chore : log 제거 * refactor : coop Api 분리 * refactor : image_url로 변수명 수정 * refactor : image_url로 변수명 수정 * chore : menuId로 변수명 구체화 * chore : V$ -> V3 버전 수정 * remove : repository 계층 트랜잭션 제거(service 계층 처리) * feat : 허용되지 않은 권한에 대한 테스트 추가 * refactor : update 수정(setter 이용) * refactor : 테스트 - 허용되지않은 권한 수정 * refactor : getById() 추가 * chore : Optional 검증을 위해 findById() -> getById() 수정 * remove() : DynamicUpdate 제거 * chore : 충돌 해결 * chore : 파일 위치 수정 * feat : 알림 구독 엔티티 생성 * feat: flyway 테이블 추가 * feat: API 형태 추가 * feat: 알림구독 추가, 삭제 컨트롤러 작성 * feat: 레포지토리 추가중.. * feat : delete api 수정 * feat: post 수정 중 .. * feat: GET 요청 반환값 만들기 * feat: 안정화 * feat: notification 팩토리 추가 * feat: getNotification 테스트코드 작성중 * test: 테스트 작성 중.. * feat: getNotification 테스트코드 작성완료 * feat: 스네이크케이스로 변경 후 디바이스토큰 추가 테스트코드 작성 * feat: post반환값 수정 * feat : deleteNotification 테스트 코드 추가 * feat : NotificationApiTest 완성 * remove : 알림 구독 관련 테스트 제거 * fix: 상속 받는 예외 클래스 수정 * fix: BIGINT -> UNSIGNED INT 수정 * test: koinAppUrl 추가 * style: 라인포맷팅 * test: .log().all() 제거 --------- Co-authored-by: Hyeonsu Lee <127578418+20HyeonsuLee@users.noreply.github.com> Co-authored-by: songsunkook <songsunkook@gmail.com> Co-authored-by: HyeonsuLee <leehyeonsu4888@naver.com> Co-authored-by: 최준호 <junho5336@gmail.com> * feat: 메뉴 변경 여부 추가 (#250) * feat: 메뉴 변경 확인 추가 * refactor: 테스트 수정 * feat: flyway 컬럼 추가 * refactor: V4에서 V5로 변경 * rename: V5에서 V6으로 변경 * Revert "feat: 메뉴 변경 여부 추가 (#250)" (#273) This reverts commit f8458cc. * refactor: 메뉴 변경 여부 컬럼명 통일 (#274) * feat: 메뉴 변경 확인 추가 * refactor: 테스트 수정 * feat: flyway 컬럼 추가 * refactor: V4에서 V5로 변경 * rename: V5에서 V6으로 변경 * refactor: 필드명 변경 * refactor: 필드명 변경 * feat: 버스 남은 시간 조회 - 시내버스 (#239) * feat: API 골격 작성 * feat: mongoDB 의존성 추가 * feat: 응답 객체 작성 * feat: 예외 추가 * feat: 엔티티 작성 * feat: 미운행인 버스 필터링 * feat: enum 추가 * feat: 남은 시간 계산 로직 작성 * feat: 가장 최근 버스까지 남은 시간 응답 구현 * feat: 미운행 요일 제외 로직 추가 * feat: 버스 정거장 검증로직 적용 및 중복되는 시간 제외 * refactor: enum 객체를 사용하도록 수정 * refactor: 등하교 방향 기준 탐색에서 정류장 탐색으로 로직 변경 * remove: 미사용 코드 제거 BusCourse.isRunning() * refactor: 테스트 mocking 위해 Clock 사용하도록 수정 * test: 테스트컨테이너 mongoDB 추가 * test: 셔틀 버스 조회 테스트 추가 * feat: 시내버스 Open API 호출 로직 작성 * feat: 방향 판단 로직 추가 * fix: 호출 예외 수정 * feat: json 파싱 로직 작성 1. json 파싱 로직 작성 2. 결과 코드 검증 3. 예외 추가 (500 반환) * feat: API 골격 작성 * feat: mongoDB 의존성 추가 * feat: 응답 객체 작성 * feat: 예외 추가 * feat: 엔티티 작성 * feat: 미운행인 버스 필터링 * feat: enum 추가 * feat: 남은 시간 계산 로직 작성 * feat: 가장 최근 버스까지 남은 시간 응답 구현 * feat: 미운행 요일 제외 로직 추가 * feat: 버스 정거장 검증로직 적용 및 중복되는 시간 제외 * refactor: enum 객체를 사용하도록 수정 * refactor: 등하교 방향 기준 탐색에서 정류장 탐색으로 로직 변경 * remove: 미사용 코드 제거 BusCourse.isRunning() * refactor: 테스트 mocking 위해 Clock 사용하도록 수정 * test: 테스트컨테이너 mongoDB 추가 * test: 셔틀 버스 조회 테스트 추가 * docs: API 문서에 파라미터 정보 명시 * remove: 불필요 주석 제거 * refactor: 누락된 Transactional 추가 * fix: mocking 변수 선언부 이동 * remove: 불필요 주석 제거 * refactor: 도착 시각 패턴 불일치 시 예외처리 * refactor: 반복문을 stream으로 변환 * refactor: 엔티티 필드로 List 사용 시 미리 초기화 * refactor: String 비교 시 NPE 예방 * refactor: 불필요한 매핑 메서드 제거 * refactor: 변수 분리 * rename: 변수명 수정 * fix: 충돌 해결 * refactor: Builder 생성자 접근제어자 수정 * refactor: enum 필드 간소화 * fix: 충돌 해결 * feat: 버스번호 필터링 * feat: 시내버스 조회 정보를 레디스 캐시로 저장 * feat: 버전 최신화 메서드 작성 * refactor: Pair를 사용하지 않도록 수정 * feat: ApiType 추가 - BusType과 BusOpenApiRequester 타입 맵핑을 위함 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: 주석 추가 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: record로 수정 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: 시내버스 캐시 저장 정보 변경 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * feat: 시내버스 버스 번호 반환 추가 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * feat: 버스 타입별 로직 분기 추가 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: 캐싱 정보 추가 - 남은 시간만 캐싱 -> 버스 도착 정보 통째로 캐싱 - api url 수정 - 버전 정보 저장 - forEach -> for Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: 시내버스 남은 시간 계산 로직 추가 - 버스 캐싱 정보 수정 - 시내버스 남은 시간 계산 로직 추가 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: 응답 객체 수정 - 기존: "now_bus": {"bus_number": null, ...} - 수정: "now_bus": null Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: 주석 추가 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: 응답 객체 null 체크 추가 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * feat: test 작성 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: 빈 배열 반환하도록 수정 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: 도착시간순 정렬 추가 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: 시간 차 수정 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: null 필터링 수정 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: updated_at 변경 로직 수정 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: .log().all() 삭제 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: switch문 if문으로 변경 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: 복붙 문제 수정 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * style: 컨벤션 적용 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * rename: BusInfoCache -> BusCache 이름 변경 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: 주석 수정 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * style: 컨벤션 적용 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: .log().all() 삭제 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> * refactor: 패키지 분리 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> * refactor: 상수, 검증 로직 분리 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> * refactor: 파일 이름 수정 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> * refactor: Bus.java 삭제와 구조 변경 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> * refactor: 시내버스 남은 시간 - Redis 테스트 수정 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> * refactor: 시내버스 남은 시간 - OpenApi 테스트 중간 작성 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> * refactor: 변수명 변경 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> * feat: 시내버스 남은 시간 - OpenApi 테스트 추 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> * refactor: 시내버스 테스트 수정 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: Constant 클래스 enum으로 변경 * refactor: 캐시 만료 시간 분리 * Revert "refactor: Constant 클래스 enum으로 변" This reverts commit 5dcbf975cccbbd276dfa5e8dbbb52864eca6c209. * Revert "Revert "refactor: Constant 클래스 enum으로 변"" This reverts commit f321b7eb60f0d5426a1fef4d839c4861a08198a2. * refactor: Constant 클래스 네이밍 변경 * feat: 버스 번호 필터링 추가 - 400, 402, 405 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> * chore: 충돌 해결 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> * style: 줄바꿈 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> * refactor: 중괄호 제거 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> * test: OPEN_API_KEY 추가 * refactor: 리뷰 반영 * refactor: 디버그 코드 삭제 --------- Co-authored-by: songsunkook <songsunkook@gmail.com> Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * feature: 상점 사장님 관련 추가, 수정, 삭제 / 일반유저 모든 상점 조회 api작성 (#270) * feat: delete Api 구현 * feat: 테스트코드 작성 * feat: 메뉴생성요청 dto작성 * feat: 상점 사장님 추가, 수정 / 일반 유저 모든상점조회 api 작성 * feat: 라인 포맷팅 * feat: 테스트코드 로그 제거 * feat: 피드백 반영 1차 * chore: 디버깅용 출력 제거 * chore: 리뷰반영 2 --------- Co-authored-by: HyeonsuLee <leehyeonsu4888@naver.com> * feat: spring actuator 추가 (#276) Co-authored-by: Jang Jun Young <police0022@naver.com> * fix: 상점 생성 서버에러(@Valid 추가) (#281) Co-authored-by: HyeonsuLee <leehyeonsu4888@naver.com> * refactor: 조회 방식 수정 (#284) * feat: 상점 모든 카테고리 조회 api작성 (#282) * feat: delete Api 구현 * feat: 테스트코드 작성 * feat: 메뉴생성요청 dto작성 * feat: 상점 사장님 추가, 수정 / 일반 유저 모든상점조회 api 작성 * feat: 라인 포맷팅 * feat: 테스트코드 로그 제거 * feat: 피드백 반영 1차 * chore: 디버깅용 출력 제거 * chore: 리뷰반영 2 * feat: 모든 상점 카테고리 조회 api작성 * chore: 충돌해결 * chore: 충돌해결 --------- Co-authored-by: HyeonsuLee <leehyeonsu4888@naver.com> * �fix: student Entity 필드 수정 (#286) * refactor: #283 작업 revert * refactor: UserIdentity Ordinal로 수정 * refactor: 로그아웃, 탈퇴에 대한 권한 수정 (#288) * refactor: 이메일 인증 요청 dto 값 수정 (#290) * refactor: dto 네이밍 수정 * refactor: 사용자 성별 반환값 ordinal로 수정 * feat: lecture blank error (#293) * fix: 공백 문자 처리 * chore: log 제거 * style: 코드 효율적으로 전환 * fix: timetables 반환값 수정 (#294) * refactor: 반환값 형태 수정 * test: 테스트 값 수정 * test: 주석 제거 * refactor: trim -> strip 으로 변경 * test: 로그 제거 * fix/291 get lecture blank error reopen (#296) * fix: 공백 문자 처리 * chore: log 제거 * style: 코드 효율적으로 전환 * fix: 완전 비어 있는 리스트 처리 * chore: 우선 복구 * feat: 가게 이벤트 정보 조회 (#298) * feat: stage logback 적용 (#300) * feat: 로깅 추가 * feat: dev profile에만 추가되도록 구성 * feat: slack 알림발송 로직 추가 * refactor: 테스트용 코드 제거 * feat: slack 알림 메시지 수정 (#306) * feat: slack 알림 메시지 수정 * feat: 피드백 반영 * feat: 피드백 반영2 --------- Co-authored-by: Jang Jun Young <police0022@naver.com> * fix : userGender null 값 처리 추가 (#304) * feat: get member (#311) * feat: Get /members 구현 * feat: 사용자 단건조회 구현 * test: 테스트 구성 * fix: 없는 상점 조회 시 예외 발생 (#313) * fix: 없는 상점 조회시 예외 던지기 * chore: 개행 삭제 * chore: 라인포맷팅 --------- Co-authored-by: HyeonsuLee <leehyeonsu4888@naver.com> * fix: 연관관계 수정 (#315) Co-authored-by: HyeonsuLee <leehyeonsu4888@naver.com> * fix: 상점 사장님 회원가입 에러 수정 (#316) * fix: 연관관계 수정 * fix: 연관관계 수정 --------- Co-authored-by: HyeonsuLee <leehyeonsu4888@naver.com> * Fix: bearerToken 추출 오류 수정 (#309) * refactor: 강의시간 문자열 분리 수정 * refactor: BEARER_TYPE 수정 * refactor: 수정 * feat: is_event값 추가 (#302) * test: add JsonAssertions * feat: isEvent 값 추가 * test: 인자 변수로 수정 * test: 테스트 수정 * feat: 시외버스 API 구현 (#317) * feat: API 골격 작성 * feat: mongoDB 의존성 추가 * feat: 응답 객체 작성 * feat: 예외 추가 * feat: 엔티티 작성 * feat: 미운행인 버스 필터링 * feat: enum 추가 * feat: 남은 시간 계산 로직 작성 * feat: 가장 최근 버스까지 남은 시간 응답 구현 * feat: 미운행 요일 제외 로직 추가 * feat: 버스 정거장 검증로직 적용 및 중복되는 시간 제외 * refactor: enum 객체를 사용하도록 수정 * refactor: 등하교 방향 기준 탐색에서 정류장 탐색으로 로직 변경 * remove: 미사용 코드 제거 BusCourse.isRunning() * refactor: 테스트 mocking 위해 Clock 사용하도록 수정 * test: 테스트컨테이너 mongoDB 추가 * test: 셔틀 버스 조회 테스트 추가 * feat: 시내버스 Open API 호출 로직 작성 * feat: 방향 판단 로직 추가 * fix: 호출 예외 수정 * feat: json 파싱 로직 작성 1. json 파싱 로직 작성 2. 결과 코드 검증 3. 예외 추가 (500 반환) * feat: API 골격 작성 * feat: mongoDB 의존성 추가 * feat: 응답 객체 작성 * feat: 예외 추가 * feat: 엔티티 작성 * feat: 미운행인 버스 필터링 * feat: enum 추가 * feat: 남은 시간 계산 로직 작성 * feat: 가장 최근 버스까지 남은 시간 응답 구현 * feat: 미운행 요일 제외 로직 추가 * feat: 버스 정거장 검증로직 적용 및 중복되는 시간 제외 * refactor: enum 객체를 사용하도록 수정 * refactor: 등하교 방향 기준 탐색에서 정류장 탐색으로 로직 변경 * remove: 미사용 코드 제거 BusCourse.isRunning() * refactor: 테스트 mocking 위해 Clock 사용하도록 수정 * test: 테스트컨테이너 mongoDB 추가 * test: 셔틀 버스 조회 테스트 추가 * docs: API 문서에 파라미터 정보 명시 * remove: 불필요 주석 제거 * refactor: 누락된 Transactional 추가 * fix: mocking 변수 선언부 이동 * remove: 불필요 주석 제거 * refactor: 도착 시각 패턴 불일치 시 예외처리 * refactor: 반복문을 stream으로 변환 * refactor: 엔티티 필드로 List 사용 시 미리 초기화 * refactor: String 비교 시 NPE 예방 * refactor: 불필요한 매핑 메서드 제거 * refactor: 변수 분리 * rename: 변수명 수정 * fix: 충돌 해결 * refactor: Builder 생성자 접근제어자 수정 * refactor: enum 필드 간소화 * fix: 충돌 해결 * feat: 버스번호 필터링 * feat: 시내버스 조회 정보를 레디스 캐시로 저장 * feat: 버전 최신화 메서드 작성 * refactor: Pair를 사용하지 않도록 수정 * feat: ApiType 추가 - BusType과 BusOpenApiRequester 타입 맵핑을 위함 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: 주석 추가 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: record로 수정 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: 시내버스 캐시 저장 정보 변경 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * feat: 시내버스 버스 번호 반환 추가 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * feat: 버스 타입별 로직 분기 추가 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: 캐싱 정보 추가 - 남은 시간만 캐싱 -> 버스 도착 정보 통째로 캐싱 - api url 수정 - 버전 정보 저장 - forEach -> for Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: 시내버스 남은 시간 계산 로직 추가 - 버스 캐싱 정보 수정 - 시내버스 남은 시간 계산 로직 추가 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: 응답 객체 수정 - 기존: "now_bus": {"bus_number": null, ...} - 수정: "now_bus": null Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: 주석 추가 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: 응답 객체 null 체크 추가 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * feat: test 작성 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: 빈 배열 반환하도록 수정 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: 도착시간순 정렬 추가 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: 시간 차 수정 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: null 필터링 수정 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: updated_at 변경 로직 수정 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: .log().all() 삭제 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: switch문 if문으로 변경 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: 복붙 문제 수정 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * style: 컨벤션 적용 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * rename: BusInfoCache -> BusCache 이름 변경 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: 주석 수정 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * style: 컨벤션 적용 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: .log().all() 삭제 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> * refactor: 패키지 분리 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> * refactor: 상수, 검증 로직 분리 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> * refactor: 파일 이름 수정 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> * refactor: Bus.java 삭제와 구조 변경 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> * refactor: 시내버스 남은 시간 - Redis 테스트 수정 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> * refactor: 시내버스 남은 시간 - OpenApi 테스트 중간 작성 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> * refactor: 변수명 변경 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> * feat: 시내버스 남은 시간 - OpenApi 테스트 추 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> * refactor: 시내버스 테스트 수정 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * refactor: Constant 클래스 enum으로 변경 * refactor: 캐시 만료 시간 분리 * Revert "refactor: Constant 클래스 enum으로 변" This reverts commit 5dcbf975cccbbd276dfa5e8dbbb52864eca6c209. * Revert "Revert "refactor: Constant 클래스 enum으로 변"" This reverts commit f321b7eb60f0d5426a1fef4d839c4861a08198a2. * refactor: Constant 클래스 네이밍 변경 * feat: 버스 번호 필터링 추가 - 400, 402, 405 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> * chore: 충돌 해결 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> * style: 줄바꿈 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> * refactor: 중괄호 제거 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> * test: OPEN_API_KEY 추가 * refactor: 리뷰 반영 * feat : 인수인계용 커밋 * feat: 시외버스기능구현 Co-authored-by: dradnats1012 <herjebi1012@gmail.com>, Choon0414 <hyunn815@naver.com> * feat: 시외버스 리팩터링 Co-authored-by: dradnats1012 <herjebi1012@gmail.com>, Choon0414 <hyunn815@naver.com> * test: 테스트 수정 Co-authored-by: dradnats1012 <herjebi1012@gmail.com>, Choon0414 <hyunn815@naver.com> * test: 테스트 수정 Co-authored-by: dradnats1012 <herjebi1012@gmail.com>, Choon0414 <hyunn815@naver.com> * test: 테스트 수정 Co-authored-by: dradnats1012 <herjebi1012@gmail.com>, Choon0414 <hyunn815@naver.com> * test: 테스트 제거 * refactor: 미사용 메소드 제거 * refactor: 네이밍 수정 * refactor: 네이밍 수정 * refactor: 네이밍 수정 * refactor: ttl 선언방식 수정 * refactor: 불필요한 추상화 제거 * refactor: 네이밍 수정 * style: 공백제거 * style: 코드 포맷팅 * style: 미사용 메소드 제거 * style: 미사용 클래스 제거 * refactor: API 사용 수정 * refactor: 클래스명 수정 --------- Co-authored-by: songsunkook <songsunkook@gmail.com> Co-authored-by: 박성빈 <psb106305@gmail.com> Co-authored-by: dradnats1012 <herjebi1012@gmail.com> Co-authored-by: Choon0414 <hyunn815@naver.com> * fix: 사장님 가게정보 수정 오류 (#319) * refactor: cascade 옵션 수정 * refactor: flush 추가 * fix: menu 정보 수정 시에도 flush 추가 (#320) * refactor: cascade 옵션 수정 * refactor: flush 추가 * refactor: flush 추가 * refactor: 식단 품절, 변경 여부 (Boolean -> 시간)으로 변경 (#310) * refactor: flyway 추가 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> * refactor: Boolean에서 LocalDateTime으로 변경 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> * test: Boolean에서 LocalDateTime으로 수정 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> * test: DisplayName 수정 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> * refactor: 품절 취소로직 추가 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> * refactor: DTO NotNull Valid 추가 Co-authored-by: dradnats1012 <herjebi1012@gmail.com> --------- Co-authored-by: dradnats1012 <herjebi1012@gmail.com> * feat: 유저 권한 조회 API 작성 (#322) * feat: 유저 권한 조회 API 작성 * test: 테스트 작성 * refactor: 식단 조회 API DTO 필드명 변경 (#324) * fix: @param추가 (#327) Co-authored-by: HyeonsuLee <leehyeonsu4888@naver.com> * feat: 학생 회원가입 (#271) * feat: 에외 추가 * feat: 이메일 폼 추가 * feat: 골격 추가 * feat: 서비스 로직 구현 * feat: 인증 완료 폼 * feat: 회원 등록 폼 타임리프 설정 * feat: 이메일 인증 실패 폼 * feat: Controller 추가 * feat: 인증 완료시 isAuthed true 변경 * feat: 학번 학부 인증 및 UserIdentity->ordinal로 변경 * feat: 학생 회원가입 dto * feat: Service 구현 * feat: 토큰으로 부터 찾는 메서드 Repository에 추가 * feat: authToken 암호화 Util * feat: koreatehc.ac.kr 도메인 검증 * feat: 학번과 학부 검증 * feat: 슬랙 알림 * feat: 학생 이메일 요청, 가입 이벤트 * feat: 토큰 유효기간 설정 DateUtil * feat: 잘못된 학번 형식 예외 * feat: 이메일 인증 검증 dto * feat: 회원가입 이메일 인증 폼 데이터 * refactor: contorller 메서드 수정 * feat: 테스트 작성 * refactor: 컨벤션 맞게 수정 * refactor: log().all() 제거 * refactor: 컨벤션에 맞게 수정 * refactor: authToken passwordEncoder를 통해 암호화 * refactor: 학번 검증 수정 * refactor: 학번 검증 수정 * refactor: 메서드 네이밍수정 * refactor: 필드 controller->service 이동 * refactor: 유효하지 않은 데이터 400으로 반환 * refactor: 라인포맷팅 * refactor: 라인포맷팅 * feat: 에러코드반환수정 및 학번검증 테스트 추가 * refactor: 오류 해결 * refactor: 폼 반환 로직 수정 * refactor: 라인포맷팅 * refactor: 날짜 관련 로직 수정 * refactor: 호스트주소 추출 어노테이션으로 변경 * refactor: 학부,학번 검증 로직 변경 * refactor: 비밀번호 example 중복 제거 * refactor: 폼 반환 service 로직 수정 * refactor: 호스트 주소 어노테이션 적용 * refactor: 머지 형식에 맞게 수정 * refactor: 최신 전공 형식에 맞게 수정 * refactor: InvalidDataException -> IllegalException 변경 * refactor: 개행 수정 * refactor: camelCase로 수정 * refactor: 테스트명 수정 * refactor: expiredAt clock인자 받게 수정 * refactor: authToken UUID로 수정 * refactor: LocalTimeStringConverter 이름을 LocalTimeToHHmmStringConverter로 변경 * refactor: User객체 날짜 관련 속성LocalTimeToHHmmStringConverter 이용 * refactor: 학번 검증 수정 * refactor: host어노테이션 이름 변경 * refactor: LocalDate관련 수정 * refactor: 라인포맷팅 * refactor: LocalDate관련 클래스이름 수정 * refactor: URL얻어오는 코드 수정 * refactor: AuthResult orElse -> orElseGet * refactor: LocalTimeAttributeConverter 원래대로 수정 * refactor: 라인포맷팅 * refactor: orElseGet 수정 * feat: 신규상점 생성시 기본 카테고리 추가 (#331) * fix: @param추가 * feat: 신규 상점 생성 시 기본 메뉴 카테고리 추가 * chore: 라인포맷팅 --------- Co-authored-by: HyeonsuLee <leehyeonsu4888@naver.com> * fix: AttributeConverter 오류 수정 (#335) * feat: FCM 발송방식 수정 (#333) * docs: github workflow 설정 추가 * refactor: 알림 발송형태 수정 * refactor: 컬럼명 수정 * feat: isEvent 추가 (#337) * fix: shopId null일때 처리하기 (#341) * feat: 비밀번호 변경 로직 구현 (#342) * feat : 수정을 위한 1차 커밋 * feat : mailForm 생성 및 DTO, mail html 추가 * feat: controller 작성 * feat: webConfig 추가 * feat: resetToken전달하도록 수정 * feat: 이메일 전송 * feat: 버튼 누를 때 기능 * chore : 권한 static import * chore : LocalDateTime 형식으로 수정 및 미사용 파일 삭제 * feat: 비밀번호 초기화 구현 * docs: hidden 추가 * style: 개행제거 --------- Co-authored-by: duehee <149302959+duehee@users.noreply.github.com> Co-authored-by: daheeParkk <qkrekgml7414@naver.com> * fix : JsonProperty 삭제 (#345) * fix : Mixed Content 헤더 추가 (#347) * fix: attributeConverter milli second 길이가 두개인 경우도 반영 (#348) * fix: mili second 길이가 두개인 경우도 반영 * fix: authExpiredAt 시간 변경 * chore: datetimeformatter 변수명 변경 * Fix: attribute converter 오류 수정 (#349) * refactor: 회원가입 인증 토큰 만료기간 수정 * fix: AttributeConverter 오류 수정 * fix : 삭제된 userGender null 로직 추가 (#357) * feat: 이벤트 추가, 수정, 삭제, 베너조회 api 작성 (#350) * fix: @param추가 * feat: 신규 상점 생성 시 기본 메뉴 카테고리 추가 * chore…

* feat : DTO 추가 * feat : 유저 수정 추가 * test : log.all() 삭제 및 422번 오류 삭제 * chore : update 수정 및 사용하지 않는 Request 객체 삭제 * refactor : update 이름 수정 * refactor : DTO major 라인 포맷팅 * chore : bulider() 미사용 및 update 로직 수 * chore : 리뷰 반영 및 학부/학번 404 -> 400번 오류로 수정 * chore : ApiResponse 수정 * chore : 리뷰 반영 * chore : withDetail 수정 * chore : 리뷰 반영(코드 스타일 및 메소드 분리) * chore : Local 내 회원 정보 조회 오류 확인 추가 * chore : 삼항 연산자 제거 * chore : Response department null 처리 추가 * chore : 변수 수정 및 Gender null 처리 추가 * chore : 변수 수정 및 department null 처리 추가 * chore : DB 학과 한글 저장 * chore : DB 저장 체크(Enumerated 삭제) * chore : DB 저장 체크(Enumerated 삭제 및 model 이름 수정) * chore : ENUM 사용하도록 재수정 * fix : ENUM -> STRING 사용 * chore : 검증 ENUM으로 이동 * chore : ENUM 영어 변수로 수정 * chore : ENUM 후행 쉼표 추가 (cherry picked from commit 0751eb7)

* feat : DTO 추가 * feat : 유저 수정 추가 * test : log.all() 삭제 및 422번 오류 삭제 * chore : update 수정 및 사용하지 않는 Request 객체 삭제 * refactor : update 이름 수정 * refactor : DTO major 라인 포맷팅 * chore : bulider() 미사용 및 update 로직 수 * chore : 리뷰 반영 및 학부/학번 404 -> 400번 오류로 수정 * chore : ApiResponse 수정 * chore : 리뷰 반영 * chore : withDetail 수정 * chore : 리뷰 반영(코드 스타일 및 메소드 분리) * chore : Local 내 회원 정보 조회 오류 확인 추가 * chore : 삼항 연산자 제거 * chore : Response department null 처리 추가 * chore : 변수 수정 및 Gender null 처리 추가 * chore : 변수 수정 및 department null 처리 추가 * chore : DB 학과 한글 저장 * chore : DB 저장 체크(Enumerated 삭제) * chore : DB 저장 체크(Enumerated 삭제 및 model 이름 수정) * chore : ENUM 사용하도록 재수정 * fix : ENUM -> STRING 사용 * chore : 검증 ENUM으로 이동 * chore : ENUM 영어 변수로 수정 * chore : ENUM 후행 쉼표 추가

duehee added 3 commits March 20, 2024 15:56

feat : DTO 추가

17d1958

Merge branch 'develop' of https://github.com/BCSDLab/KOIN_API_V2 into…

688ef63

… feature/23-put-user-student-me

feat : 유저 수정 추가

2cd4809

duehee added the 기능 새로운 기능을 개발합니다. label Mar 23, 2024

duehee requested review from daheeParkk, kwoo28 and seongjae6751 March 23, 2024 12:38

duehee self-assigned this Mar 23, 2024

feat : 충돌 해결

a05f899

kwoo28 approved these changes Mar 24, 2024

View reviewed changes

seongjae6751 requested changes Mar 24, 2024

View reviewed changes

duehee added 2 commits March 25, 2024 13:32

test : log.all() 삭제 및 422번 오류 삭제

9a15e02

chore : update 수정 및 사용하지 않는 Request 객체 삭제

9948155

songsunkook requested changes Mar 25, 2024

View reviewed changes

refactor : update 이름 수정

9d03c7b

duehee added the Team User 유저 팀에서 작업할 이슈입니다 label Mar 25, 2024

songsunkook requested changes Mar 26, 2024

View reviewed changes

duehee added 2 commits March 26, 2024 18:18

refactor : DTO major 라인 포맷팅

fa1cace

chore : bulider() 미사용 및 update 로직 수

7c6aeb8

duehee requested review from seongjae6751 and songsunkook March 26, 2024 15:37

Choi-JJunho requested changes Mar 27, 2024

View reviewed changes

duehee added 3 commits March 27, 2024 14:25

chore : 리뷰 반영 및 학부/학번 404 -> 400번 오류로 수정

bbc8eb9

feat : 충돌 해결

14acf71

chore : ApiResponse 수정

bd1f71d

seongjae6751 approved these changes Mar 27, 2024

View reviewed changes

Choi-JJunho requested changes Mar 27, 2024

View reviewed changes

duehee added 2 commits March 28, 2024 15:06

chore : 리뷰 반영

7e4005b

chore : withDetail 수정

08664cd

duehee requested a review from Choi-JJunho March 28, 2024 06:14

songsunkook requested changes Mar 28, 2024

View reviewed changes

Choi-JJunho approved these changes Mar 29, 2024

View reviewed changes

duehee added 7 commits March 29, 2024 12:52

chore : 리뷰 반영(코드 스타일 및 메소드 분리)

a631ea5

feat : 충돌 해결

a0daa35

chore : Local 내 회원 정보 조회 오류 확인 추가

302e7dd

chore : 삼항 연산자 제거

f7df525

chore : Response department null 처리 추가

7cb4e31

chore : 변수 수정 및 Gender null 처리 추가

91c2de4

chore : 변수 수정 및 department null 처리 추가

d64505c

duehee added 4 commits March 30, 2024 01:51

chore : DB 학과 한글 저장

07cf7a7

chore : DB 저장 체크(Enumerated 삭제)

95accd0

chore : DB 저장 체크(Enumerated 삭제 및 model 이름 수정)

25db4ab

chore : ENUM 사용하도록 재수정

b8e5867

duehee requested a review from songsunkook March 29, 2024 17:37

songsunkook requested changes Mar 30, 2024

View reviewed changes

fix : ENUM -> STRING 사용

b592981

songsunkook requested changes Mar 30, 2024

View reviewed changes

duehee added 3 commits March 30, 2024 15:52

chore : 검증 ENUM으로 이동

fcea148

chore : ENUM 영어 변수로 수정

c13775c

chore : ENUM 후행 쉼표 추가

fe6adfb

songsunkook approved these changes Mar 30, 2024

View reviewed changes

duehee merged commit 0751eb7 into develop Mar 30, 2024

duehee deleted the feature/23-put-user-student-me branch March 30, 2024 07:22


		import in.koreatech.koin.global.exception.DataNotFoundException;

		public class StudentDepartmentNotValidException extends DataNotFoundException {


		import in.koreatech.koin.global.exception.DataNotFoundException;

		public class StudentNumberNotValidException extends DataNotFoundException {

	student.getDepartment().toString(),
	student.getDepartment().getValue(),

	user.userInfoUpdate(studentUpdateRequest.nickname(), studentUpdateRequest.name(),
	user.update(studentUpdateRequest.nickname(), studentUpdateRequest.name(),

		public class StudentDepartmentNotValidException extends IllegalArgumentException {
		private static final String DEFAULT_MESSAGE = "학생의 전공 형식이 아닙니다.";

	EMPLOYMENT("고용서비스정책학과");
	EMPLOYMENT("고용서비스정책학과"),
	;

	@Schema(description = "이메일 주소 \n", example = "koin123@koreatech.ac.kr")
	@Schema(description = "이메일 주소", example = "koin123@koreatech.ac.kr")

	@Schema(description = "이름 \n", example = "최준호")
	@Schema(description = "이름", example = "최준호")

Comments

Conversation

duehee commented Mar 23, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🔥 연관 이슈

🚀 작업 내용

💬 리뷰 중점사항

Uh oh!

github-actions bot commented Mar 23, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Unit Test Results

Uh oh!

kwoo28 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

C

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

R

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

C

Uh oh!

duehee Mar 25, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

C

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

C

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

seongjae6751 left a comment

Choose a reason for hiding this comment

Uh oh!

seongjae6751 Mar 24, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

songsunkook Mar 25, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

R

Uh oh!

Choose a reason for hiding this comment

Uh oh!

songsunkook left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

C

Uh oh!

Choose a reason for hiding this comment

C

duehee commented Mar 23, 2024 •

edited

Loading

github-actions bot commented Mar 23, 2024 •

edited

Loading

duehee Mar 25, 2024 •

edited

Loading

seongjae6751 Mar 24, 2024 •

edited

Loading

songsunkook Mar 25, 2024 •

edited

Loading

duehee Mar 26, 2024 •

edited

Loading