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

[Feature] 셰어 공개상담 개발 요청 사항 반영 / react-query 반영 #300

Merged
merged 65 commits into from
Jun 15, 2024

Conversation

kyuhho
Copy link
Contributor

@kyuhho kyuhho commented Jun 13, 2024

Checklist


  • 올바른 브랜치에 PR을 보내도록 설정하였나요?
  • PR의 라벨을 올바르게 달았나요?

Description


  • 셰어 공개상담 개발 요청 사항 반영했습니다.
  • react-query 반영하고 기존 axios 코드 deprecated 처리하였습니다.

Related Issues


#294

To Reveiwer


  • axios 변경사항 꼭 확인 부탁드립니다.
  • 디자인 꺠지는 부분이 없는지 테스트 한번 부탁드립니다.
  • 고칠 부분 코드리뷰 부탁드립니다!

@kyuhho kyuhho added API API 관련 Feature 기능 개발 Style 스타일링 Priority-1 🔥 priority 1 Stage-2/code-review-request work done and CI passed labels Jun 13, 2024
@kyuhho kyuhho self-assigned this Jun 13, 2024
Copy link

netlify bot commented Jun 13, 2024

Deploy Preview for candid-semolina-d0db42 ready!

Name Link
🔨 Latest commit 26f242a
🔍 Latest deploy log https://app.netlify.com/sites/candid-semolina-d0db42/deploys/666dac12e1e1150008bd56de
😎 Deploy Preview https://deploy-preview-300--candid-semolina-d0db42.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@rmdnps10
Copy link
Contributor

0521 셰어-공개상담변경사항도 같이 반영해주실 수 있을까요?

스크린샷 2024-06-14 오후 6 26 42

Copy link
Contributor

@rmdnps10 rmdnps10 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다~!

queryKey: ['infiniteGetPostsCutsomersResponse', isChecked],
queryFn: async ({ pageParam }) =>
await getPostsCutsomers(pageParam).then((res) => {
console.log(res.data);
Copy link
Contributor

Choose a reason for hiding this comment

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

console.log 는 삭제해주시면 좋을 거 같습니다!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

반영하였습니다! b6edd86

}
};
export const axiosGet = async (url: string, params?: any) =>
await instance.get(url, { params });
Copy link
Contributor

Choose a reason for hiding this comment

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

try-catch문과 이별하기 좋아요 ㅎㅎ

Copy link
Contributor

Choose a reason for hiding this comment

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

이 방식이 더 깔끔한 거 같습니다~ Goood

Comment on lines +62 to +70
const {
data: openConsults,
isLoading,
fetchNextPage,
hasNextPage,
isFetching,
isFetchingNextPage,
isLoadingError,
} = useInfiniteQuery<GetPostsCutsomersResponse[]>({
Copy link
Contributor

Choose a reason for hiding this comment

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

리액트 쿼리를 통한 무한 스크롤 구현하신 코딩 패턴 확인했습니다~!

Copy link
Contributor

@rmdnps10 rmdnps10 Jun 14, 2024

Choose a reason for hiding this comment

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

공개 상담 단건 조회 페이지를 비롯한 상담 상세 페이지에서 게시물 좋아요 , 북마크, 댓글 좋아요 혹은 아이콘을 눌렀을 때 숫자 부분 width가 달라져서 아이콘이 밀리는 거 같습니다!

좋아요/스크랩 숫자 부분을 좋아요 자릿수에 따라 width를 지정해두는 것도 방법일 거 같습니다!

2024-06-14.7.39.58.mov

Copy link
Contributor Author

Choose a reason for hiding this comment

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

앗 그런 이슈가 있군요!
저는 layout을 구성할 때 width, height의 직접적인 선언을 지양하는 것이 반응형에 좋은 practice라고 생각하여 위와 같이 구현하였습니다!

여러 부분에서 사용하는 태그이기 때문에 다음 작업에서 따로 마이그레이션하는 방향으로 하겠습니다!

@kyuhho
Copy link
Contributor Author

kyuhho commented Jun 15, 2024

0521 셰어-공개상담변경사항도 같이 반영해주실 수 있을까요?

스크린샷 2024-06-14 오후 6 26 42

아직 해당 부분 description이 확정되지 않은 것 같아 따로 이슈 생성하여 작업하겠습니다!

@rmdnps10 rmdnps10 merged commit 031c353 into dev Jun 15, 2024
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API API 관련 Feature 기능 개발 Priority-1 🔥 priority 1 Stage-2/code-review-request work done and CI passed Style 스타일링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants