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] 어드민 유저 검색 관련 #136

Merged
merged 3 commits into from
Aug 23, 2024
Merged

Conversation

darkdulgi
Copy link
Collaborator

📝 작업 내용

  • 어드민 유저 조회 페이지에서 이제 성명, 전화번호, 이벤트 frameId의 필드값으로도 검색이 가능합니다.
  • 어드민 유저 조회 페이지에서 검색 쿼리(검색 문자열, 필드)를 주소로 띄워 뒤로가기로 이전 검색결과를 확인할 수 있도록 했습니다.

@darkdulgi darkdulgi added the feat 기능 구현 label Aug 23, 2024
@darkdulgi darkdulgi self-assigned this Aug 23, 2024
Copy link
Collaborator

@lybell-art lybell-art left a comment

Choose a reason for hiding this comment

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

수고하셨습니다.

<Route path="/comments" element={<CommentsPage />} />
<Route path="/comments/:eventId" element={<CommentsIDPage />} />
Copy link
Collaborator

Choose a reason for hiding this comment

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

왜 /comments/:eventId의 순서를 바꾸셨나요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

처음에 /comments의 자식 route로 :eventId를 넣으려 했는데 저희 설계상 맞지 않아서 그냥 뺐는데 그 잔재입니다

const [page, setPage] = useState(1);
const data = useQuery(
"admin-users",
() =>
fetchServer(
`/api/v1/admin/event-users?page=${page - 1}&search=${searchString}&field=${category}&size=15`,
`/api/v1/admin/event-users?page=${page - 1}&search=${searchParams.get("search") ?? ""}&field=${searchParams.get("field") ?? "userName"}&size=15`,
Copy link
Collaborator

Choose a reason for hiding this comment

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

searchParams을 잘 쓰시는군요

@lybell-art lybell-art merged commit 16095fa into dev Aug 23, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat 기능 구현
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants