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 : search api 연결 #62

Merged
merged 6 commits into from
Mar 10, 2024
Merged

feat : search api 연결 #62

merged 6 commits into from
Mar 10, 2024

Conversation

Ubinquitous
Copy link
Member

Issue Number

close #61

What

검색 api를 연결했습니다.

How

ScreenShot

스크린샷 2024-03-10 오전 5 32 15

@Ubinquitous Ubinquitous added the feat 새로운 기능 label Mar 9, 2024
@Ubinquitous Ubinquitous self-assigned this Mar 9, 2024
Copy link
Member

@jyh071116 jyh071116 left a comment

Choose a reason for hiding this comment

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

LGTM😎

Copy link
Member

@5ewon 5ewon left a comment

Choose a reason for hiding this comment

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

Thank you for your work

<input
onChange={({ target: { value } }) => setKeyword(value)}
Copy link
Member

Choose a reason for hiding this comment

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

event객체 구조분해할당 방식을 사용하니 원하는 데이터만 깔끔하게 사용할 수 있네요. 이벤트 객체는 사용해본적이 없었는데 신기하네요.

Copy link
Member Author

Choose a reason for hiding this comment

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

저도 이벤트에서 value값만 쓰이는데 이벤트 객체 자체를 뽑는게 어색하다고 생각해서 이렇게 사용하고있습니다!

@Ubinquitous Ubinquitous merged commit 23024af into main Mar 10, 2024
2 checks passed
@Ubinquitous Ubinquitous deleted the feat/search branch November 15, 2024 16:25
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.

검색 페이지 api 연결
3 participants