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

[feat] #127 useQuery hook #127

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

[feat] #127 useQuery hook #127

wants to merge 4 commits into from

Conversation

nayoung3669
Copy link
Member

1. 무슨 이유로 코드를 변경했나요?

리액트쿼리를 하나의 커스텀훅으로 분리했습니다. 메인페이지와 마이페이지에서 사용하며 제네릭타입을 이용합니다.

2. 어떤 위험이나 장애를 발견했나요?


3. 관련 스크린샷을 첨부해주세요.


4. 완료 사항


5. 추가 사항

close #124

@nayoung3669 nayoung3669 self-assigned this Apr 2, 2024
@eunji0714
Copy link
Member

확인했습니다! custom hook을 사용해서 코드를 간결화하여 훨씬 간결해진 것 같아요 :)

@One-HyeWon
Copy link
Member

확인했습니다! 하나로 합쳐진 코드가 훨씬 간결해서 좋은 것 같아요!

Copy link

@minsoo-web minsoo-web left a comment

Choose a reason for hiding this comment

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

안녕하세요! 우연히 깃허브 둘러보다가 방문하게 되어서,, 올라와있는 PR 보고 리뷰 남겨보게 되었습니다..!

src/app/_hooks/useCustomQuery.ts Show resolved Hide resolved
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.

[feat] #124 useQuery hook
4 participants