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

feat ✨ follow api 추가 #412

Merged
merged 26 commits into from
Feb 20, 2024
Merged

feat ✨ follow api 추가 #412

merged 26 commits into from
Feb 20, 2024

Conversation

niamu01
Copy link
Contributor

@niamu01 niamu01 commented Dec 25, 2023 •
edited
Loading

@niamu01 niamu01 added the enhancement New feature or request label Dec 25, 2023
@niamu01 niamu01 self-assigned this Dec 25, 2023
Base automatically changed from fix/netwhatInDailyEvalRecord to dev December 27, 2023 04:21
@niamu01 niamu01 force-pushed the feat/follow branch 3 times, most recently from aa268cc to 638c1cd Compare December 29, 2023 12:22
@niamu01 niamu01 marked this pull request as ready for review December 29, 2023 13:19
@niamu01
Copy link
Contributor Author

niamu01 commented Dec 29, 2023

날라갈 뻔한 커밋 체리픽 완...! 👍

jpham005
jpham005 requested changes Dec 30, 2023
View reviewed changes
Copy link
Member

@jpham005 jpham005 left a comment

Choose a reason for hiding this comment

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

caching 활용이 전혀 안되어 있어 이 부분은 근본적으로 고쳐야할 것으로 보이고,
db index 도 관리해야합니다. 지금은 아무것도 없더군요.

graphql model 도 잘 이해가 안되는 부분들이 있는데,
프론트와 같이 보면서 실제로 어떻게 사용하는지 살펴보고 고치던가 해야겠습니다.

app/src/api/cursusUser/cursusUser.service.ts Outdated Show resolved Hide resolved
app/src/follow/follow.resolve.ts Outdated Show resolved Hide resolved
app/src/follow/follow.resolve.ts Outdated

const pubSub = new PubSub();

@UseFilters(HttpExceptionFilter)
Copy link
Member

Choose a reason for hiding this comment

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

global filter 한번 찾아보세요.
guard 도 마찬가지

적용은 안해도 괜찮을 것 같습니다.

app/src/follow/follow.resolve.ts Outdated Show resolved Hide resolved
app/src/follow/follow.service.ts Outdated Show resolved Hide resolved
app/src/follow/follow.service.ts Outdated Show resolved Hide resolved
app/src/follow/model/follow.model.ts Outdated Show resolved Hide resolved
app/src/follow/follow.resolve.ts Outdated Show resolved Hide resolved
app/src/follow/model/follow.model.ts Outdated Show resolved Hide resolved
app/src/follow/model/follow.model.ts Outdated Show resolved Hide resolved
@niamu01 niamu01 marked this pull request as draft January 20, 2024 08:49
@niamu01 niamu01 marked this pull request as ready for review February 20, 2024 13:07
@niamu01
feat: ✨ FollowerList 추가
23a8eb7 
- #405

feat: ✨ MakeFollow: 프론트 테스트용 임시 함수 추가

- #405

refactor: ♻️ 변수명 수정 및 함수 분리

login -> target, FollowListByMe -> FollowList, FollowUserList -> FollowListWithCount, followUser->followLser, cursusUserService에 유저 아이디 찾는 함수 추가, 중복되는 함수 분리

- #405

refactor: ♻️ checkFollowingStatus 함수 활용하도록 변경

- #405
@niamu01
refactor: ♻️ 리스트에 자신이 있는 경우 isFollowing을 null로 반환
3ea4d7e 
- #405
@niamu01
feat: ✨ follow/unfollow에 대해 성공했을 시 subscription 추가
bad161f 
- #405

refactor: ♻️ 프론트 테스트용 임시 함수 리팩토링

- #405

refactor: ♻️ type-only import 추가

- #405

fix: 🐛 mutation이 아닌 쿼리에 대해서 mutation -> query 변경

- #405
@niamu01
refactor: ♻️ 팔로우 리스트 최신순으로 반환하도록 변경
2ac0c50 
- #406
@niamu01
refactor: ♻️ 팔로우 리스트에 limit 추가
6e85655 
- #406
@niamu01
feat: ✨ follow list pagination 구현
0de57a1 
- #406

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

- #406
@niamu01
feat: ✨ 자신이 팔로우한 여부 확인 쿼리 추가
4403cee 
- #406
@niamu01
refactor: ♻️ list 쿼리에도 sort의 defaultvalue 추가
877225e 
- #406

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

- #406

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

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

- #406

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

- #406

fix: 🐛 변수 대입을 안하고 있었던 부분 수정 (...)
@niamu01
refactor: ♻️ aggregate -> find~~ 로 변경, 필요한 데이터만 받도록 select 사용
b034841 
- #406
@niamu01
refactor: ♻️ followingStatus -> isFollowing 으로 변경 및 중복되는 부분 삭제
3442797 
- #406

style: 🎨 lint
@niamu01
fix: 🐛 pubSub.publish 함수에 await 추가
8661352 
- #406

revert: 🔥 의미없는 코드 삭제
@niamu01
refactor: ♻️ 한개만 찾는 요소에 대하여 find -> findOne로 변경
1d3d5c3 
- #406

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

- #406

revert: 🔥 follower, following List 삭제

- #406

refactor: ♻️ findOne 할 때 select 추가
@niamu01
refactor: ♻️ evalLog의 fieldExtractor의 커서 검사 사항 추가
7f5667c
@niamu01
refactor: ♻️ pagination cursor base -> index base
f268b91 
- #406
@niamu01
refactor: ♻️ unionType -> throw Exception
df90c93 
- #406
@niamu01
refactor: ♻️ targetLogin -> targetId 로 변경
1652b9b 
- #406

refactor: ♻️ targetLogin -> targetId 로 변경

- #406

fix: 🐛 변수명 오류 수정
@niamu01
feat: ✨ 테스트 용 temp_데이터베이스 사용
80ba12d 
- #413
@niamu01
feat: ✨ follower, following 구분해 캐시 형태 결정
9f14267 
- #413

fix: 🐛 follow create 안하던 부분 수정
@niamu01
refactor: ♻️ db와 cache에 저장되는 타입이 다름을 고려 후 로직 변경
ee2e835 
db에 요청: userId, followId 로 follower/following 리스트를 만든 후, userPreview와 followAt 을 추가해 cache와 같은 타입으로 변환 / cache에 요청, db요청 이후: {userPreview, followAt}[] 에 로그인 한 유저에 맞춰 isFollowing 추가 후 반환

- #413

fix: 🐛 존재하는 유저인지 검사 후 db에 write

- #413

fix: 🐛 실수 수정 following->follower

프론트와 디버깅 중 발견했습니다

- #413

feat: ✨ preload 임시 적용

이후 중복되는 코드를 정리해야합니다

- #413

refactor: ♻️ 주석 삭제 및 중복되는 함수 분리

- #413

fix: 🐛 팔로우 리스트 캐시 업데이트시 배열의 push를 사용하여 무조건 뒤에 붙던 일 임시 해결

sort를 밖에서 하는 방식으로 변경할 예정입니다

- #413
@niamu01
feat: 🐛 isFollowing 도 cache에서 확인하도록 변경
264b7f1 
- #413
@niamu01
feat: 🐛 cache된 리스트를 반환할 때도 sort한 후 반환하도록 추가
295e1ff 
- #413
@niamu01
refactor: ♻️ follow시 push하기 때문에 set을 다시 호출하던 부분 삭제, 못 찾는 경우와 삭제 실패가 같…
bf1abd5 
…은 404를 throw하여 find하지 않고 바로 delete하여 로직 줄이기

- #413
@niamu01
refactor: ♻️ 타입, 변수명 변경
1a64608 
FollowList[] -> Follow[], cache되던 형태의 FollowListCacheType를 Follow로 변경하고 원래 Follow였던 최종 반환 형태를 MyFollow로 변경, 이에 따른 dto파일명이나 pagination 타입도 함께 맞추어 변경

- #413
@niamu01
Copy link
Contributor Author

niamu01 commented Feb 20, 2024

#414 풀리퀘에 이어서 함께 머지 합니당

@niamu01 niamu01 merged commit cfc4ae7 into dev Feb 20, 2024
@niamu01 niamu01 deleted the feat/follow branch February 20, 2024 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jpham005 jpham005 jpham005 requested changes

Assignees

@niamu01 niamu01

Labels
enhancement New feature or request
Projects
42Stat
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@niamu01 @jpham005