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

Feature 9/추가 api 구현 #10

Merged
merged 13 commits into from
Apr 13, 2024
Merged

Feature 9/추가 api 구현 #10

merged 13 commits into from
Apr 13, 2024

Conversation

Ryeolee
Copy link
Contributor

@Ryeolee Ryeolee commented Apr 7, 2024

API 구현

  1. 내 채소 등록 API
  2. 내 채소 루틴 추가 API
  3. 내 채소 성장 일기 추가 API
  4. 내 채소 리스트 API
  5. 성장일기 추가 여부 확인 API
  6. 메인페이지 성장일기 조회 API

DB Table수정

  1. Routine엔티티 notify 컬럼 추가 -> 알림 울림 여부를 구별하기 위해서

Ryeolee and others added 10 commits April 5, 2024 15:19
      2. 내 채소 일기 등록 API 미완성
MyVeggie.java -> diary cascade ALL 설정
MyVeggieGardenService 에 채소 찾는 메서드 추가
MyVeggieDiaryService 로직 findById() 이후 addDiary()로 변경
…e_9/추가-API-구현

# Conflicts:
#	src/main/java/com/modernfarmer/farmusspring/domain/myveggiegarden/service/MyVeggieDiaryService.java
       2. 내 채소 일기 추가 여부 확인 API 완료
@Ryeolee Ryeolee added the enhancement New feature or request label Apr 7, 2024
@MinchoGreenT
Copy link
Contributor

알림 설정의 경우 일괄적으로 알림 on/off만 존재하는걸로 아는데, 각 채소가 알림 여부를 가지고 있는건 맞지 않아 보입니다.

  • 이거랑 오타 또 몇개 보여서.. My를 Myy로 쓴다던가.. 수정해서 올려주세요~

@Ryeolee
Copy link
Contributor Author

Ryeolee commented Apr 8, 2024

혹시 내 채소 컬럼에 알림이 없고, 내 채소의 루틴에 알림 여부가 있습니다.
그리고 어제 회의 끝나고 말한
if(checkNullDiaryList(diaryList)) {
return BaseResponseDto.of(MyVeggieGardenErrorCode.NOT_FOUND_DIARY, null);
}
이 부분 다시 확인을 해봤는데 diaryList형식으로 db에서 조회하고 이 중 첫번째 데이터만 각 데이터별로 리스폰스로 주는데, 이러면 빈배열로 줄 수가 없어서 이렇게 처리했습니다!
일단
if(checkNullDiaryList(diaryList)) {
return BaseResponseDto.of(MyVeggieGardenSuccessCode.NOT_FOUND_DIARY, null);
}
ErrorCode라는 enum이 어색해서
SuccessCode에 적용해놨습니다.
그리고 오타 수정했습니다!

@MinchoGreenT
Copy link
Contributor

혹시 내 채소 컬럼에 알림이 없고, 내 채소의 루틴에 알림 여부가 있습니다. 그리고 어제 회의 끝나고 말한 if(checkNullDiaryList(diaryList)) { return BaseResponseDto.of(MyVeggieGardenErrorCode.NOT_FOUND_DIARY, null); } 이 부분 다시 확인을 해봤는데 diaryList형식으로 db에서 조회하고 이 중 첫번째 데이터만 각 데이터별로 리스폰스로 주는데, 이러면 빈배열로 줄 수가 없어서 이렇게 처리했습니다! 일단 if(checkNullDiaryList(diaryList)) { return BaseResponseDto.of(MyVeggieGardenSuccessCode.NOT_FOUND_DIARY, null); } ErrorCode라는 enum이 어색해서 SuccessCode에 적용해놨습니다. 그리고 오타 수정했습니다!

코멘트 늦게봐서 죄송합니다,, 근데 혹시 일기가 찾아지지 않는 에러인데 왜 SuccessCode에 넣으신거죠...?

@Ryeolee
Copy link
Contributor Author

Ryeolee commented Apr 10, 2024

찾아지지 않는다고 해서 에러는 아니라서요?? 일기를 안쓰면 없는거라서요!

@MinchoGreenT
Copy link
Contributor

그러면 에러명을 Not Found 와 같은 내용으로 쓰면 안될 것 같습니다(메세지 내용도 해당 일기가 존재하지 않는다고 반환하는데, 채소가 가지고 있는 일기 데이터가 없는 상황이니 메세지가 잘못 된 것 같습니다). 또한� 조회에 성공했는데 코드를 따로 짜서 반환할 필요는 없어보여요.

@Ryeolee
Copy link
Contributor Author

Ryeolee commented Apr 10, 2024

흠 그럼 별다른 체킹 함수없이 바로 내보낼까요?

@MinchoGreenT
Copy link
Contributor

흠 그럼 별다른 체킹 함수없이 바로 내보낼까요?

넹 바로 갑시다 수정하고 머지 고고하시죠

@Ryeolee Ryeolee merged commit 8d73054 into dev Apr 13, 2024
1 check passed
@Ryeolee Ryeolee deleted the feature_9/추가-API-구현 branch April 13, 2024 02:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants