-
Notifications
You must be signed in to change notification settings - Fork 77
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(view): cluster node를 펼쳤을 때 avatar 보여주기 #404
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!
<div className="avatar-message"> | ||
{authSrcMap && ( | ||
<Author | ||
name={author.names.toString()} | ||
src={authSrcMap[author.names.toString()]} | ||
/> | ||
)} | ||
<span className="message">{message}</span> | ||
</div> |
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.
추후에,
map을 param으로 받는 방식도 괜찮지만,
authSrcMap[author.names.toString()]
을 인자로 받거나, contextAPI등에서 데이터를 가져오는 방식도 고려해보면 좋을 것 같습니다.
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.
사용자가 보기에 더욱 깔끔해서 보기 좋습니다!!
This reverts commit f0bc40e.
This reverts commit f0bc40e.
Related issue
close #362
Result
Work list
기존 commit message 좌측에 author의 avatar를 추가했습니다.
Discussion
conflict를 해결하다가 이전 PR이 close 돼버려 다시 PR 올렸습니다.