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.

Sign up for GitHub

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

Jump to bottom

회원 스토리 조회 API 추가 #149

Merged
merged 6 commits into from
Mar 24, 2022
Merged

회원 스토리 조회 API 추가 #149

merged 6 commits into from
Mar 24, 2022

Conversation

seonpilKim
Copy link
Member

@seonpilKim seonpilKim commented Mar 23, 2022 •
edited
Loading

변경 사항

MemberStory 저장 방식 변경

  • 스토리마다 업로드 시간이 다르면, 각각 24시간 후 삭제가 되어야 하므로, 스토리마다 ttl을 따로 적용하는 방식으로 변경해야 함.
  • memberId를 secondary index로 설정하여 여러 개의 스토리를 MemberStory에 각각 담아 저장

p.s) 회원 스토리 업로드 여부 조회 방식 변경할 예정. #147

회원 스토리 조회 API 추가

  • 응답 데이터 中 seenId 의미
    • 로그인한 회원이 조회하려는 회원 스토리 중 가장 마지막에 조회한 스토리의 PK
    • 회원 스토리는 처음부터 순서대로 조회하는 방식 -> 3번쨰까지 조회했으면 3번쨰 스토리의 PK가 저장됨
    • seenId는 회원 스토리 방문자 여부로 판단하여 적용
      • 회원 스토리 방문자가 하나라도 있으면, 가장 마지막 방문한 스토리의 PK 저장
      • 방문자가 없다면, 아직 해당 회원의 스토리를 조회하지 않은 상황임 -> 가장 마지막 스토리의 PK 저장
  • 프론트단에서 seenId와 회원 스토리 중 가장 마지막 pk를 비교
    • 다를 경우, seenId 다음 스토리부터 조회
    • 같을 경우, 처음 스토리부터 조회

AuthUtil 메소드 추가

  • 각 Repository에서 불필요하게 MemberRepository를 의존하지 않도록, AuthUtil에서만 MemberRepository를 의존해서 로그인한 회원 엔티티를 반환하는 메소드를 추가 함.
  • 의존성을 가능한 없애야 유지보수하기도 좋고, 테스트 코드 작성도 깔끔하게 가능
  • 이후로도 의존성을 없애는 리팩토링 위주로 먼저 작업할 필요가 있음

해결 이슈

@seonpilKim seonpilKim added the ✈enhancement New feature or request label Mar 23, 2022
@seonpilKim seonpilKim requested a review from vectorch9 March 23, 2022 21:13
@seonpilKim seonpilKim self-assigned this Mar 23, 2022
vectorch9
vectorch9 approved these changes Mar 24, 2022
View reviewed changes
Copy link
Member

@vectorch9 vectorch9 left a comment

Choose a reason for hiding this comment

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

exists류 쿼리 최적화 랑 AuthUtil 사용하는건 저도 패키지 작업 이후에 적용하겠습니다

@seonpilKim seonpilKim merged commit 4bc66c0 into develop Mar 24, 2022
@seonpilKim seonpilKim deleted the Feature/Story branch March 24, 2022 07:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vectorch9 vectorch9 vectorch9 approved these changes

Assignees

@seonpilKim seonpilKim

Labels
✈enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

회원 스토리 조회 API MemberStory 저장 방식 변경
2 participants
@seonpilKim @vectorch9