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

♻ 팔로우 기능 api 수정 #406

Closed
8 tasks done
niamu01 opened this issue Dec 19, 2023 · 0 comments
Closed
8 tasks done

♻ 팔로우 기능 api 수정 #406

niamu01 opened this issue Dec 19, 2023 · 0 comments
Assignees
Labels
enhancement New feature or request

Comments

@niamu01
Copy link
Contributor

niamu01 commented Dec 19, 2023

231218 @42sungwook 님과 회의로 새롭게 결정된 부분과 잘못 구현한 사항을 수정하려 합니다.
231220 @yoopark 님에게 전달 및 추가 회의..!
231228 @42sungwook 님이 만드신 페이지 보구 얘기 나눈 결과 자신이 팔로우 한 유저인지만 알려주는 쿼리 따로 분리하기로 결정 (다른 유저의 정보 페이지에서 사용)
240105 @42sungwook follower/ingList 삭제(pagination으로 대체 전달), pagination 방식 커서 -> 페이지 기반으로 변경
이외에 pr 댓글에 남겨져 있던 의견 이야기해보기

  • follow 성공과 unfollow 성공 subscription에서 구분하는 법 -> 구분 x
  • 팔로우 최신순으로 반환해 주기
  • 팔로우 리스트 반환 시 limit 추가
  • 팔로우 리스트 pagination
  • 자신이 팔로우 했는지 여부 쿼리 따로 제공
  • follower/ingList 삭제
  • pagination 방식 cursor -> index 기반으로 변경
  • UnionType -> exception (일단 404로 통일)

+) evalLog cursor validate check

@niamu01 niamu01 added this to 42Stat Dec 19, 2023
@niamu01 niamu01 converted this from a draft issue Dec 19, 2023
@niamu01 niamu01 moved this from 🔖 Ready to 🏗 In progress in 42Stat Dec 19, 2023
@niamu01 niamu01 self-assigned this Dec 19, 2023
@niamu01 niamu01 added the enhancement New feature or request label Dec 19, 2023
niamu01 added a commit that referenced this issue Dec 29, 2023
niamu01 added a commit that referenced this issue Dec 30, 2023
niamu01 added a commit that referenced this issue Jan 5, 2024
niamu01 added a commit that referenced this issue Feb 20, 2024
- #406

fix: 🐛 following 관련 함수에서 사용한 follower -> following 수정

- #406

fix: 🐛 파일명 수정 resolve -> resolver

style: 🎨 변수 선언 시 필요 없는 타입 어노테이션 제거

- #406

fix: 🐛 필요없는 if 검사 삭제

- #406

fix: 🐛 변수 대입을 안하고 있었던 부분 수정 (...)
niamu01 added a commit that referenced this issue Feb 20, 2024
- #406

revert: 🔥 의미없는 코드 삭제
niamu01 added a commit that referenced this issue Feb 20, 2024
- #406

refactor: ♻️ 팔로우 여부에 대해 나 자신 예외처리 제거, input을 login에서 id로 변경

- #406

revert: 🔥 follower, following List 삭제

- #406

refactor: ♻️ findOne 할 때 select 추가
niamu01 added a commit that referenced this issue Feb 20, 2024
- #406

refactor: ♻️ targetLogin -> targetId 로 변경

- #406

fix: 🐛 변수명 오류 수정
niamu01 added a commit that referenced this issue Feb 20, 2024
- #406

revert: 🔥 임시 함수 삭제 및 주석 삭제

- #406
niamu01 added a commit that referenced this issue Feb 20, 2024
- #406

fix: 🐛 following 관련 함수에서 사용한 follower -> following 수정

- #406

fix: 🐛 파일명 수정 resolve -> resolver

style: 🎨 변수 선언 시 필요 없는 타입 어노테이션 제거

- #406

fix: 🐛 필요없는 if 검사 삭제

- #406

fix: 🐛 변수 대입을 안하고 있었던 부분 수정 (...)
niamu01 added a commit that referenced this issue Feb 20, 2024
- #406

revert: 🔥 의미없는 코드 삭제
niamu01 added a commit that referenced this issue Feb 20, 2024
- #406

refactor: ♻️ 팔로우 여부에 대해 나 자신 예외처리 제거, input을 login에서 id로 변경

- #406

revert: 🔥 follower, following List 삭제

- #406

refactor: ♻️ findOne 할 때 select 추가
niamu01 added a commit that referenced this issue Feb 20, 2024
- #406

refactor: ♻️ targetLogin -> targetId 로 변경

- #406

fix: 🐛 변수명 오류 수정
@niamu01 niamu01 closed this as completed Feb 20, 2024
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in 42Stat Feb 20, 2024
niamu01 added a commit that referenced this issue Apr 10, 2024
refactor: ♻️ 팔로우 리스트에 limit 추가

- #406
niamu01 added a commit that referenced this issue Apr 10, 2024
- #406

revert: 🔥 임시 함수 삭제 및 주석 삭제

- #406
niamu01 added a commit that referenced this issue Apr 10, 2024
fix: 🐛 following 관련 함수에서 사용한 follower -> following 수정
fix: 🐛 파일명 수정 resolve -> resolver
style: 🎨 변수 선언 시 필요 없는 타입 어노테이션 제거
fix: 🐛 필요없는 if 검사 삭제
fix: 🐛 변수 대입을 안하고 있었던 부분 수정 (...)
refactor: ♻️ aggregate -> find~~ 로 변경, 필요한 데이터만 받도록 select 사용
refactor: ♻️ followingStatus -> isFollowing 으로 변경 및 중복되는 부분 삭제
style: 🎨 lint

- #406
niamu01 added a commit that referenced this issue Apr 10, 2024
- #406

revert: 🔥 의미없는 코드 삭제
niamu01 added a commit that referenced this issue Apr 10, 2024
refactor: ♻️ 팔로우 여부에 대해 나 자신 예외처리 제거, input을 login에서 id로 변경
revert: 🔥 follower, following List 삭제
refactor: ♻️ findOne 할 때 select 추가
refactor: ♻️ evalLog의 fieldExtractor의 커서 검사 사항 추가

- #406
niamu01 added a commit that referenced this issue Apr 10, 2024
refactor: ♻️ targetLogin -> targetId 로 변경
fix: 🐛 변수명 오류 수정
feat: ✨ 테스트 용 temp_데이터베이스 사용

- #406
- #413
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
Status: Done
Development

No branches or pull requests

1 participant