-
Notifications
You must be signed in to change notification settings - Fork 2
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
게시글, 댓글에 프로필사진 보여주기 #78
Conversation
CommunityProfilePicture(model = null) // TODO: 서버에서 프로필이미지 내려주면 반영하기 | ||
CommunityProfilePicture(model = post.profilePicture) |
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.
지금은 프로필이 비어 있으면 서버에서 기본 프로필을 내려 주지 않아서 클라에 따로 리소스 수정하고 구현해야 한다는 말이지?
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.
서버에 문의해보니까 기본 프로필이미지를 내려주는 대신 기본인지 아닌지 Bool을 내려줄려는 거 같아서 기본일때 user의 프로필 이미지를 R.drawable에서 가져다 쓰는 방법으로 해야할 듯? UserAccount(닉네임이랑 프로필이미지 바꾸는 화면)에서도 그렇게 할려고
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.
아하... 클라들이 귀찮게 됐네
CommunityProfilePicture(model = null) // TODO: 서버에서 프로필이미지 내려주면 반영하기 | ||
CommunityProfilePicture(model = post.profilePicture) |
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.
서버에 문의해보니까 기본 프로필이미지를 내려주는 대신 기본인지 아닌지 Bool을 내려줄려는 거 같아서 기본일때 user의 프로필 이미지를 R.drawable에서 가져다 쓰는 방법으로 해야할 듯? UserAccount(닉네임이랑 프로필이미지 바꾸는 화면)에서도 그렇게 할려고
추가된 점
기타