기록하기 API #22
기록하기 API #22
Conversation
* 자바 버전업 되면서 aws s3 모듈중 일부 모듈이 제거되어 생기는 wa경고 , 해당 추가한 모듈을 사용권장하여 추가
* Spring Boot 기본값으로 이미 주입되는 값들이라 삭제
| @@ -15,7 +15,6 @@ public class Member { | |||
| @GeneratedValue(strategy = GenerationType.IDENTITY) | |||
| private Long id; | |||
|
|
|||
| @Column(nullable = false) | |||
There was a problem hiding this comment.
이름은 null이어도 되고 심지어 지금은 아예 필요가 없습니다 서버에서 랜덤으로 만들어주는 닉네임을 쓸거라서요
There was a problem hiding this comment.
클라한테 지역을 입력받는데 얘네말고 다른값들이 들어오는걸 방지하기위함입니다 Enum한번 보심 될것같아유
| /** | ||
| * | ||
| */ |
There was a problem hiding this comment.
옹 이친구들은 지워도 좋을 거 같애용 혹시 뭐 적으려다가 깜빡하셨나영
|
|
||
| @NotBlank(message = "명소 이름은 필수 입력 사항입니다.") | ||
| @Pattern(regexp = "^[가-힣]+$", message = "명소 이름은 한글만 입력 가능합니다.") | ||
| @Size(max = 50, message = "명소 이름은 50자 이내여야 합니다.") |
There was a problem hiding this comment.
이미지 세장까지 업로드 가능하게 해야하는데, 그 제한이 어디에 있나요 !?
혹시나 있다면 제가 못찾는 거 같아요 알려주세요 !!!!
There was a problem hiding this comment.
오 추가하지 않았었는데 검증이 필요해보이네요
| public static MemberRegion of(Member member, Region region, int visitCount) { | ||
| public static MemberRegion of(Member member, Region region) { | ||
| return MemberRegion.builder() | ||
| .member(member) | ||
| .region(region) | ||
| .visitCount(visitCount) | ||
| .visitCount(1) |
There was a problem hiding this comment.
MemberRegion 객체는 특정 지역에 방문했을때 최초로 생성이되고, 이후에는 visitCount가 add되는 형태라서 이 객체를 생성했을땐 1번 방문했다는거고, 임의의 visitCount를 받아 생성할 일은 없다고 판단했어요
| @NotNull(message = "날짜는 필수 입력 사항입니다.") | ||
| LocalDate localDate |
There was a problem hiding this comment.
이거 포맷 프로토타입에서 기록할 때 날짜 형식은 2024년7월21일로 되어 있고, 기록 확인할 때는 24.07.21 되어 있네용 ?!
이거에 관련해서 클라이언트 분들이랑 얘기했던 거 같은데 결론이 기억 안나네요 ,,?ㅎ
* /service/AppleOAuthService.java uses or overrides a deprecated API. 경고가 있었음
What is this PR? 🔍
방문기록 저장 API 구현