-
Notifications
You must be signed in to change notification settings - Fork 5
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
refactor:: yunNote 반영 리팩토링 #276
Conversation
@@ -1,4 +1,4 @@ | |||
package com.kernel360.main.controller; | |||
package com.kernel360.main.enumset; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
enum 타입 패키지 이동 후 메서드 public 부여 확인 했습니다.
public record PasswordDto( | ||
@NotBlank(message = "비밀번호는 비어 있을수 없습니다.") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이건 저희 로그상에서만 확인 가능한 메세지를 내보내주는 역할인가 보네요
@@ -11,6 +11,8 @@ spring: | |||
format_sql: true | |||
dialect: org.hibernate.dialect.PostgreSQLDialect | |||
ddl-auto: validate | |||
naming: | |||
physical-strategy: org.hibernate.boot.model.naming.PhysicalNamingStrategyStandardImpl |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@column 애노테이션 제거 후 자동 매핑 처리를 위한 옵션 확인 했습니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
리팩터링 고생하셨습니다~
@@ -12,6 +12,8 @@ spring: | |||
format_sql: true | |||
dialect: org.hibernate.dialect.PostgreSQLDialect | |||
ddl-auto: validate | |||
naming: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hibernate 가 자동으로 이름 붙여주는 부분 확인하였습니다
public record PasswordDto( | ||
@NotBlank(message = "비밀번호는 비어 있을수 없습니다.") | ||
@Size(min = 8, max = 16, message = "비밀번호는 8~16글자입니다.") | ||
@Pattern(regexp = "^(?=.*[0-9])(?=.*[a-zA-Z])(?=\\S+$).{8,20}$", message = "8~16자의 영문 대/소문자, 숫자, 특수문자를 사용해 주세요.") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍🏻
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
확인했습니다~!
확인 요청드린 코멘트만 봐주세요~
@Size(min = 8, max = 16, message = "비밀번호는 8~16글자입니다.") | ||
@Pattern(regexp = "^(?=.*[0-9])(?=.*[a-zA-Z])(?=\\S+$).{8,20}$", message = "8~16자의 영문 대/소문자, 숫자, 특수문자를 사용해 주세요.") | ||
// @Pattern(regexp = "^(?=.*[0-9])(?=.*[a-zA-Z])(?=.*\\W).{8,16}$", message = "8~16자의 영문자(대/소문자 구분 없음), 숫자, 특수문자를 사용해 주세요.") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Size에 대한 내용과 대소문자 구분을 보면 아래 주석을 한 부분의 패턴이 맞는 것 같은데 아래 패턴을 주석처리하고 위 패턴으로 적용하신 이유가 있으실까요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
깔끔해지겠군요 ㅎㅎ
저도 이후에 만드는 건 일단 이렇게 해봐야겠어요
💡 Motivation and Context
여기에 왜 이 PR이 필요했는지, PR을 통해 무엇이 바뀌는지에 대해서 설명해 주세요
는 carInfo, washInfo entity 대상으로 테스트 삼아 바꾸어보았습니다. 데이터베이스 반영은 잘되는 것을 확인했습니다.
다른 entity 적용여부는 얘기를 해보시면 좋을 듯 합니다.
🔨 Modified
🌟 More
📋 커밋 전 체크리스트
🤟🏻 PR로 완료된 이슈
closes #270