-
Notifications
You must be signed in to change notification settings - Fork 1
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM😎
There was a problem hiding this 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)} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
event객체 구조분해할당 방식을 사용하니 원하는 데이터만 깔끔하게 사용할 수 있네요. 이벤트 객체는 사용해본적이 없었는데 신기하네요.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
저도 이벤트에서 value값만 쓰이는데 이벤트 객체 자체를 뽑는게 어색하다고 생각해서 이렇게 사용하고있습니다!
Issue Number
close #61
What
검색 api를 연결했습니다.
How
ScreenShot