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

[GGFE-177] 관리자 거래내역 특정유저 조회 기능 #931

Merged

Conversation

PHJoon
Copy link
Contributor

@PHJoon PHJoon commented Aug 14, 2023

📌 개요

  • 관리자 거래내역 특정유저 조회 기능

💻 작업사항

  • 관리자 - 거래내역, 확성기, 프로필 사용내역에서 인트라 검색해서 해당 유저 정보만 가져오는 기능 추가
  • mock api에서 hyungjpa, jeyoon, sangmipa, hyobicho 는 검색하면 데이터 나오게 만들어놨습니다.

✅ 변경로직

@PHJoon PHJoon self-assigned this Aug 14, 2023
Copy link
Member

@hyobb109 hyobb109 left a comment

Choose a reason for hiding this comment

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

유저 검색 api 호출 잘 되는 것 확인했습니다! 고생하셨습니다👍

Copy link
Member

@yoouyeon yoouyeon left a comment

Choose a reason for hiding this comment

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

검색 api 호출 잘 되고 응답 잘 보여지는 것 확인했습니다! 수고하셨습니다~!! 👍

};

const profile10: Iprofile = {
profileId: 5,
Copy link
Member

Choose a reason for hiding this comment

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

여기 id가 중복된게 있어서 중복된 키 에러가 납니다!! 수정하면 좋을 것 같습니다!!

Comment on lines 61 to 64
const res = await mockInstance.get(
`/admin/megaphones/history?intraId=${intraId}&page=${currentPage}&size=10`
);
setIntraId(intraId);
Copy link
Member

Choose a reason for hiding this comment

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

여기 흐름이 잘 이해가 안가요! mockInstance에서 intraId를 사용하는데 왜 다시 setIntraId를 하게 되는건가요???

Copy link
Contributor Author

Choose a reason for hiding this comment

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

관리자페이지 다른 곳에서 사용하는 것을 그대로 가져오면서 지우지 않은 것 같습니다. 없어도 되는 부분이라 삭제했습니다 👍👍

@PHJoon
Copy link
Contributor Author

PHJoon commented Aug 14, 2023

피드백 주신 id 키 수정, 필요없는 부분 삭제 수정했습니다!

Copy link
Member

@yoouyeon yoouyeon left a comment

Choose a reason for hiding this comment

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

확인했습니다!! 수고하셨습니다~! 😆👍

@PHJoon PHJoon merged commit f984dd9 into main Aug 16, 2023
@PHJoon PHJoon deleted the GGFE-177-관리자-거래내역-특정유저-조회-기능 branch August 16, 2023 02:20
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.

3 participants