-
Notifications
You must be signed in to change notification settings - Fork 0
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
The head ref may contain hidden characters: "feature_9/\uCD94\uAC00-API-\uAD6C\uD604"
Conversation
2. 내 채소 일기 등록 API 미완성
…e_9/추가-API-구현 # Conflicts: # src/main/java/com/modernfarmer/farmusspring/domain/myveggiegarden/service/MyVeggieDiaryService.java
2. 내 채소 일기 추가 여부 확인 API 완료
알림 설정의 경우 일괄적으로 알림 on/off만 존재하는걸로 아는데, 각 채소가 알림 여부를 가지고 있는건 맞지 않아 보입니다.
|
혹시 내 채소 컬럼에 알림이 없고, 내 채소의 루틴에 알림 여부가 있습니다. |
코멘트 늦게봐서 죄송합니다,, 근데 혹시 일기가 찾아지지 않는 에러인데 왜 SuccessCode에 넣으신거죠...? |
찾아지지 않는다고 해서 에러는 아니라서요?? 일기를 안쓰면 없는거라서요! |
그러면 에러명을 Not Found 와 같은 내용으로 쓰면 안될 것 같습니다(메세지 내용도 해당 일기가 존재하지 않는다고 반환하는데, 채소가 가지고 있는 일기 데이터가 없는 상황이니 메세지가 잘못 된 것 같습니다). 또한� 조회에 성공했는데 코드를 따로 짜서 반환할 필요는 없어보여요. |
흠 그럼 별다른 체킹 함수없이 바로 내보낼까요? |
넹 바로 갑시다 수정하고 머지 고고하시죠 |
API 구현
DB Table수정