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

Fix: CHAT-294-일기-스트릭-API-수정 #50

Merged
merged 2 commits into from
Feb 14, 2024

Conversation

Mouon
Copy link
Member

@Mouon Mouon commented Feb 14, 2024

오늘 기준으로하면 사용자의 상식적인 스트릭과 논리적 충돌있어서 하루전으로 기준 수정

요약 (Summary)

  • streakDate매서드의 조건문에서 today에서 하루 전으로 수정

변경 사항 (Changes)

  • minusDays 매서드 사용했습니다. 나머지는 거의 동일합니다.
  • 조건문이 조금 수정되었습니다. 어플의 사용자입장의 스트릭을 반영했습니다. (오늘 날짜 일기 없으면 어제 기준 스트릭, 오늘 일기 작성하면 기존 스트릭 +1 , 오늘 아무것도 안쓰면 스트릭 깨짐)

리뷰 요구사항

  • 오늘날짜에 일기가 없어도 어제 기준의 스트릭이 보여지는지 봐주세요
  • 오늘 날짜 추가되면 스트릭이 변경되는지 확인 부탁드립니다

확인 방법 (선택)

예시 쿼리

예시 주소

http://localhost:8080/diary/streak?memberId=1
INSERT INTO member (email, password, create_at, update_at, status)
VALUES ("[email protected]", "test123", "2024-01-01 13:47:36", "2024-01-01 13:47:36", "ACTIVE");

INSERT INTO tag (category, tag_name)
VALUES ("감정", "기쁨"), ("감정", "슬픔"), ("감정", "화남"), ("감정", "피곤함"), ("감정", "설렘"), ("감정", "당황"), ("감정", "무서움"),
       ("인물", "친구"), ("인물", "가족"), ("인물", "동료"), ("인물", "애인"), ("인물", "지인"),
       ("행동", "식사"), ("행동", "공부"), ("행동", "여행"), ("행동", "술"), ("행동", "영화"),
       ("행동", "수다"), ("행동", "게임"), ("행동", "업무"), ("행동", "운동"), ("행동", "휴식"), ("행동", "독서"), ("행동", "쇼핑"),
       ("장소", "식당"), ("장소", "학교"), ("장소", "회사"), ("장소", "집"), ("장소", "버스"), ("장소", "카페"), ("장소", "병원"),
       ("장소", "헬스장"), ("장소", "은행"), ("장소", "공항");


INSERT INTO diary (user_id, diary_date, title, content, create_at, update_at, status)
VALUES (1,"20240102","title1", "content1", "2024-01-02 23:59:59", "2024-01-02 23:59:59", "ACTIVE"), 
	   (1,"20240103","title2", "content2", "2024-01-03 23:59:59", "2024-01-03 23:59:59", "ACTIVE"),
       (1,"20240128","title3", "content3", "2024-01-28 23:59:59", "2024-01-28 23:59:59", "ACTIVE"), 
       (1,"20240201","title4", "content4", "2024-02-01 23:59:59", "2024-02-01 23:59:59", "ACTIVE"), 
       (1,"20240202","title5", "content5", "2024-02-02 23:59:59", "2024-02-02 23:59:59", "ACTIVE"), 
       (1,"20240203","title6", "content6", "2024-02-03 23:59:59", "2024-02-03 23:59:59", "ACTIVE"), 
       (1,"20240204","title7", "content7", "2024-02-04 23:59:59", "2024-02-04 23:59:59", "ACTIVE"), 
       (1,"20240205","title8", "content8", "2024-02-05 23:59:59", "2024-02-05 23:59:59", "ACTIVE"), 
       (1,"20240206","title9", "content9", "2024-02-06 23:59:59", "2024-02-06 23:59:59", "ACTIVE"), 
       (1,"20240207","title10", "content10", "2024-02-07 23:59:59", "2024-02-07 23:59:59", "ACTIVE"), 
       (1,"20240208","title11", "content11", "2024-02-08 23:59:59", "2024-02-08 23:59:59", "ACTIVE"), 
       (1,"20240209","title12", "content12", "2024-02-09 23:59:59", "2024-02-09 23:59:59", "ACTIVE"), 
       (1,"20240210","title13", "content13", "2024-02-10 23:59:59", "2024-02-10 23:59:59", "ACTIVE"), 
       (1,"20240211","title14", "content14", "2024-02-11 23:59:59", "2024-02-11 23:59:59", "ACTIVE"), 
       (1,"20240212","title15", "content15", "2024-02-12 23:59:59", "2024-02-12 23:59:59", "ACTIVE"),
       (1,"20240213","title15", "content15", "2024-02-13 23:59:59", "2024-02-13 23:59:59", "ACTIVE");

하루 추가용 쿼리 ( 다양한 상황 테스트)

INSERT INTO diary (user_id, diary_date, title, content, create_at, update_at, status)
VALUES (1,"20240214","title1", "content1", "2024-02-14 23:59:59", "2024-02-14 23:59:59", "ACTIVE"); 

포스트맨

스크린샷 2024-02-14 오후 5 51 28

오늘 기준으로하면 사용자 입장에서 일기스트릭을 억울하게 인정 못받는 오류있어서 수정
어제 날짜까지는 인식하는 오늘 날짜 추가시에 기존 조건 문에 걸려 버려서 수정
Copy link
Collaborator

@dainshon dainshon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

포스트맨으로 확인한 결과 잘 동작합니다:>
Uploading image.png…

return 0L;
if (diaries.isEmpty() || !diaries.get(0).getDiaryDate().toLocalDate().equals(today.minusDays(1))) {
if(diaries.get(0).getDiaryDate().toLocalDate().equals(today)){

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

혹쉬 여기에서
diaries.get(0).getDiaryDate().toLocalDate().equals(today){}안을 비워두신 이유가 있는지 궁금합니다!!

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

오늘 날짜가 조건문에 안걸리고 통과해서 다른 라인 실행하게하려고 작성했습니다. 더 나은 방법이 있겠지만, 당장 떠오르는건 저것뿐이었습니다.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

아 그렇군요! 기능상 문제는 없는것같습니다-!

@nonaninona nonaninona merged commit a8d5469 into develop Feb 14, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants